|
| 1 | +# 2025 Components Verification Summary |
| 2 | + |
| 3 | +## Status Overview |
| 4 | + |
| 5 | +Based on the verification of all 2025 components mentioned, here's the complete status: |
| 6 | + |
| 7 | +### ✅ Components Already Correctly Configured for 2025 |
| 8 | + |
| 9 | +1. **Schedule** (`/2025/schedule`) |
| 10 | + - **Status**: ✅ Correctly imports `@data/2025.json` |
| 11 | + - **Location**: `src/views/Schedule/Schedule.tsx` (line 7) |
| 12 | + - **No action needed** |
| 13 | + |
| 14 | +2. **Talks** (`/2025/talks`) |
| 15 | + - **Status**: ✅ Correctly imports `@data/2025.json` as default |
| 16 | + - **Location**: `src/views/Talks/Talks.tsx` (line 6) |
| 17 | + - **Behavior**: Uses `conferenceConfig` prop with `conferenceData` (2025) as default |
| 18 | + - **API**: Calls `useFetchTalks(conferenceConfig.edition)` with correct year |
| 19 | + - **No action needed** |
| 20 | + |
| 21 | +3. **Workshops** (`/2025/workshops`) |
| 22 | + - **Status**: ✅ Correctly imports `@data/2025.json` |
| 23 | + - **Location**: `src/views/Workshops/Workshops.tsx` (line 5) |
| 24 | + - **Note**: 2026 will not have workshops (as per requirements) |
| 25 | + - **No action needed** |
| 26 | + |
| 27 | +4. **Job Offers** (`/2025/jobOffers`) |
| 28 | + - **Status**: ✅ Uses static `jobOffers` data from `JobsData` |
| 29 | + - **Location**: `src/components/JobOffers/JobOffersList.tsx` (line 5) |
| 30 | + - **Note**: Not year-specific, uses static sponsor job data |
| 31 | + - **No action needed** |
| 32 | + |
| 33 | +### ⚠️ Component Requiring Wrapper |
| 34 | + |
| 35 | +5. **Diversity** (`/2025/diversity`) |
| 36 | + - **Status**: ⚠️ Hardcodes `@data/2026.json` |
| 37 | + - **Location**: `src/views/Diversity/Diversity.tsx` (line 3) |
| 38 | + - **Issue**: Uses `data.edition` directly (lines 95, 121) instead of accepting a prop |
| 39 | + - **Current behavior**: Shows "2026" in text |
| 40 | + - **Required action**: Create `DiversityWrapper2025` to pass 2025 data |
| 41 | + - **Note**: Both 2025 and 2026 have `diversity: false` (application form disabled) |
| 42 | + |
| 43 | +## Recommendations |
| 44 | + |
| 45 | +### Immediate Action Required |
| 46 | + |
| 47 | +**Diversity Component**: Create a wrapper for 2025 to ensure correct year is displayed. |
| 48 | + |
| 49 | +```typescript |
| 50 | +// src/2025/Diversity/DiversityWrapper2025.tsx |
| 51 | +import React, { FC } from "react"; |
| 52 | +import { Diversity } from "@views/Diversity/Diversity"; |
| 53 | +import data2025 from "@data/2025.json"; |
| 54 | + |
| 55 | +export const DiversityWrapper2025: FC = () => { |
| 56 | + return <Diversity conferenceConfig={data2025} />; |
| 57 | +}; |
| 58 | +``` |
| 59 | + |
| 60 | +**Note**: The Diversity component needs to be refactored to accept a `conferenceConfig` prop instead of hardcoding the import. |
| 61 | + |
| 62 | +### Future Considerations |
| 63 | + |
| 64 | +1. **2026 Components**: As mentioned, the following are not ready yet: |
| 65 | + - Diversity 2026: Incomplete/duplicate data is acceptable |
| 66 | + - Schedule 2026: Not ready |
| 67 | + - Talks 2026: Will be empty (API returns no records) |
| 68 | + - Workshops 2026: Will not have workshops |
| 69 | + - Job Offers 2026: Not ready (no sponsors yet) |
| 70 | + |
| 71 | +2. **Pattern Consistency**: Consider refactoring all base components to accept `conferenceConfig` props to avoid future issues when preparing new editions. |
| 72 | + |
| 73 | +## Data Configuration Summary |
| 74 | + |
| 75 | +| Component | 2025 Status | 2026 Status | Needs Wrapper | |
| 76 | +| ---------- | -------------- | --------------- | ------------- | |
| 77 | +| Schedule | ✅ Set | ❌ Not ready | No | |
| 78 | +| Talks | ✅ Set | ❌ Empty (API) | No | |
| 79 | +| Workshops | ✅ Set | ❌ No workshops | No | |
| 80 | +| Job Offers | ✅ Set | ❌ No sponsors | No | |
| 81 | +| Diversity | ✅ Set | ❌ Not ready | Yes (2025) | |
| 82 | +| Speakers | ✅ Set (fixed) | ✅ Ready | Yes (2025) | |
| 83 | +| CFP | ✅ Set (fixed) | ✅ Ready | Yes (2025) | |
| 84 | + |
| 85 | +## Testing Status |
| 86 | + |
| 87 | +- ✅ All 197 tests pass |
| 88 | +- ✅ Integration tests verify year configurations |
| 89 | +- ✅ Speakers and CFP have dedicated wrapper tests |
| 90 | +- ⏳ Diversity wrapper needs to be created and tested |
0 commit comments