[Map of Classes] Render a selected-pin popup
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 78/100
- Issue type
- Feature
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- react, typescript
- Domain
- frontend
Research direction
Create src/components/map/MeetingPopup.tsx, modelling its props and rendering on the existing src/components/schedules/DayClass.tsx example. Read that component first, then render courseCode/title, start–end, building description and optional room from a single MapMeeting, showing TBA for missing time and omitting absent building/room instead of literal null. Cover the sample case plus one missing-value case in src/components/map/MeetingPopup.test.tsx, then run pnpm test -- src/components/map/MeetingPopup.test.tsx, pnpm lint and pnpm typecheck; the issue notes #867 should land first.
Written by the indexing model from the issue text.
Description
What to build
Show class details when a map marker is selected. Include the course, meeting time and building/room information, with a readable fallback for missing values.
Files
src/components/map/MeetingPopup.tsx(new file)- Starting example: src/components/schedules/DayClass.tsx
Steps
- Render courseCode/title,start–end,building description and optional room from one plain MapMeeting.
- Render TBA for missing time; omit absent building/room rather than literal null.
- Use ordinary React text for all catalog strings.
Example
Sample/09–10/Cedar Street/101 yields those fields; null room has no empty room label.
Check it works
In src/components/map/MeetingPopup.test.tsx, check the example above and the failure or limit described in the steps. Run pnpm test -- src/components/map/MeetingPopup.test.tsx, pnpm lint and pnpm typecheck.
Before starting
- #867 — Render numbered day pins through supplied map adapter
References
Assign yourself before starting, or comment to claim the issue. Project setup.
- Dominant language
- TypeScript
- Stars
- 19
- Forks
- 94
- PR merge metrics
- No merged PRs in 30d
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from DevDogsUGA/DogDays
-
map
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
DevDogsUGA/DogDays#865 ·
-
sharing
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
DevDogsUGA/DogDays#854 ·
-
import-export
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
DevDogsUGA/DogDays#850 ·
-
import-export
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
DevDogsUGA/DogDays#842 ·
-
import-export
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
DevDogsUGA/DogDays#841 ·
All issues in DevDogsUGA/DogDays
Similar issues
-
Add: VIVE TVOpenapproved check:passed streams:add
Difficulty 1/5 Under an hour Newbie friendliness 72/100
iptv-org/iptv#55293 · 1 comment ·
Maintainers usually reply within 1 day
-
bug needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
Fission-AI/OpenSpec#2084 ·
Maintainers usually reply within 2 days
-
Contrib indemnité de départ à la retraite : faire le pont avec le simulateur sur la page génériqueOpenP1
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
SocialGouv/code-du-travail-numerique#7561 ·
Maintainers usually reply within 1 day
-
beginner bug
Difficulty 1/5 Under an hour Newbie friendliness 88/100
hiero-ledger/hiero-sdk-js#4517 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
kubernetes-sigs/headlamp#7950 ·
Maintainers usually reply within 3 days