fix: show appointment date on accompanying opportunity cards#2
Open
fix: show appointment date on accompanying opportunity cards#2
Conversation
Fix Opportunity card: Schedule need4deed-org#259
…ect-312 fix: redirect /event-page to Google Forms URL (need4deed-org#312)
ix: back button navigates to correct list page (need4deed-org#318)
…k-button Fix: back button navigates to correct list page (need4deed-org#318)
…nity-card Fix: Opportunity card: Schedule- Date time selection in case of event type need4deed-org#259
…d4deed-org#132/landing-page-layout 🎨 move landing page layout w/ placeholders need4deed-org#132
…-english-303 fix: set English as default language and persist language choice (need4deed-org#303)
Several types were renamed or restructured in the SDK bump that merged with develop, leaving the codebase in a broken state where tsc --noEmit fails on every branch: - AgentCard: use extracted `volunteerSearch` (not agent.volunteerSearch which doesn't exist on ApiAgentGetList) - helpers (Agents): add fallback for volunteerSearch being undefined - statusMaps: SDK enums (e.g. VolunteerStateEngagementType and AgentEngagementStatusType) share the same string values, making object-literal duplicate-key errors unavoidable; switch to Object.fromEntries so last-write-wins logic is preserved without compile errors - AccompanyingDetails helpers: languageToTranslate is now number in SDK, convert to string for the form - AgentContactDetails: representative → representatives[0] (field renamed to plural array in SDK) - useUpdateAgentContact / useUpdateOrganizationDetails: ApiRepresentativePatch removed from SDK; use ApiRepresentativeGet (same shape for patching)
Remove the /home segment from the authenticated dashboard route: - Delete dashboard/home/page.tsx - dashboard/page.tsx now renders DashboardHome - LoginController redirects to /dashboard after login - DashboardRoutes.Home constant updated to /dashboard Refs need4deed-org#304
…/fix/296/add-favicon 🐛 Add missing favicon.ico to fix i18next language log
…/fix/289/opportunity-card-skills-validation fix: save button and checkbox uncheck in opportunity details (need4deed-org#289)
…testimonial-render need4deed-org#336: Fixes infinite testimonial fetch
Use CSS max() to clamp the left margin to the sidebar width on narrow viewports, so the fixed navigation bar never hides the search bar or list area. Closes need4deed-org#324.
After testing, the sidebar edge was still clipping the leftmost character of list column headers. Add 16px to the clamped left margin — matching the nav bar's own horizontal padding — to give content a clear visual gap from the sidebar.
Several types were renamed or restructured in the SDK bump that merged with develop, leaving the codebase in a broken state where tsc --noEmit fails on every branch: - AgentCard: use extracted `volunteerSearch` (not agent.volunteerSearch which doesn't exist on ApiAgentGetList) - helpers (Agents): add fallback for volunteerSearch being undefined - statusMaps: SDK enums (e.g. VolunteerStateEngagementType and AgentEngagementStatusType) share the same string values, making object-literal duplicate-key errors unavoidable; switch to Object.fromEntries so last-write-wins logic is preserved without compile errors - AccompanyingDetails helpers: languageToTranslate is now number in SDK, convert to string for the form - AgentContactDetails: representative → representatives[0] (field renamed to plural array in SDK) - useUpdateAgentContact / useUpdateOrganizationDetails: ApiRepresentativePatch removed from SDK; use ApiRepresentativeGet (same shape for patching)
…-org#321 Fix: dashboard button+direct buttons to forms+ testimonials need4deed-org#321
Fix: direct buttons to forms
…-org#321 Fix: dashboard button+direct buttons to forms+ testimonials need4deed-org#321
…ng-dashboard-324 fix: prevent left nav bar from overlapping dashboard content
…der-buttons Fix: dashboard button+direct buttons to forms+ testimonials need4deed-org#321
…sues-and-wrong-imprt-path Hotfix ts issues and wrong imprt path
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…lint-github-action ci: add lint and typecheck workflow on PRs to develop
…m-sdk-bump hotfix: fix TypeScript errors introduced by SDK .73 bump
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…in SDK Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…me-304-task3 Fix/url dashboard home 304 task3
…monials-infinite-loop fix: use hook instead await for stable lang
…e-menu-from-registration-form 343: fixes menu in volunteer form
…h-to-hardcoded fix: switch to hardcoded
SDK 0.0.76 adds accompanyingDetails to ApiOpportunityGetList, so the local OpportunityWithAccompanyingDetails type extension is no longer needed. Remove it and use ApiVolunteerOpportunityGetList directly.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes need4deed-org#398
Description
Opportunity cards of type accompanying were showing no date in the "Date of appointment" field because the card was reading from availability, which is not set for accompanying opportunities. The card now reads from accompanyingDetails.appointmentDate and appointmentTime instead.
Changes:
OpportunityCardchecks if volunteerType is accompanying and reads the date fromaccompanyingDetailsinstead ofavailabilityformatAccompanyingDatehelper inOpportunityCard.helpers.tsx