Skip to content

feat(frontend): show year and role on flashcards - #30

Merged
brennenho merged 2 commits into
mainfrom
brennen/flashcard-year-role
Oct 16, 2026
Merged

brennenho merged 2 commits into
mainfrom
brennen/flashcard-year-role

Conversation

@brennenho

Copy link
Copy Markdown
Contributor

Summary

Show each member’s class standing and club role beneath their name when a flashcard is flipped.

Changes

  • Add required year and role fields for all 50 members, using CTC roster records and class standing for 2026–27.
  • Display a compact Year · Role line and include both details in screen-reader labels and announcements.
  • Validate that every roster entry has a nonempty year and role.

Testing

  • pnpm check: formatting, lint, typecheck, and all 86 tests pass.
  • Verified all 50 members have year and role values, including Julia as Junior and Serena as Senior.
  • Checked the detail layout and card navigation on desktop and 320px mobile; no text overflow or browser console errors.

@brennenho
brennenho requested review from a team as code owners October 16, 2026 22:31
@github-actions
github-actions Bot temporarily deployed to pr-30 October 16, 2026 22:32 Destroyed
@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Preview deployment

Open preview ↗

Last deployed 2026-10-16T22:37:41Z

Commit Workflow
a7ef6d1 Run #50 · attempt 1

Fallback preview

@github-actions
github-actions Bot temporarily deployed to pr-30 October 16, 2026 22:36 Destroyed
@brennenho
brennenho merged commit 2c82a4d into main Oct 16, 2026
2 checks passed
@brennenho
brennenho deleted the brennen/flashcard-year-role branch October 16, 2026 22:38
@linear-code

linear-code Bot commented Oct 16, 2026

Copy link
Copy Markdown

CTC-35

This branch was successfully deployed

No deployments
pr-30 — a7ef6d18 Deployed Oct 16, 2026 by github-actions[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant