Skip to content

Design: add region to school profile pages#219

Merged
shaynesidman merged 1 commit intomainfrom
cursor/add-region-school-profile-9183
Apr 19, 2026
Merged

Design: add region to school profile pages#219
shaynesidman merged 1 commit intomainfrom
cursor/add-region-school-profile-9183

Conversation

@tika
Copy link
Copy Markdown
Collaborator

@tika tika commented Apr 16, 2026

Issue

#102 #102

Who worked on this sprint/bug?

Cursor agent

Who did what on this sprint/bug?

Implemented the school profile UI update to include region in the school information cards.

Features Implemented

  • Added a Region card to the school profile info row.
  • Wired the school profile page to pass schoolData.region into SchoolInfoRow.
  • Preserved existing cards for Town, Implementation Model, and Data Since.

New files created

  • None.

Existing files modified

  • src/components/SchoolInfoRow.tsx
  • src/app/schools/[name]/page.tsx

Acceptance Criteria

  • Add region to school profile page cards: ✅ Completed.
  • Place region with town, instruction model, first year info cards: ✅ Completed.

Testing: how did you test?

  • Verified code changes compile syntactically and props are wired correctly.
  • Ran lint/build checks in follow-up validation step.

Features Not Implemented/Incomplete

  • No design-specific visual tweaks beyond adding the new Region card.

Bugs Discovered

  • None during implementation.

Screenshots:

  • Not captured in this run.

Tag Dan and Shayne

@danglorioso @shaynesidman

Open in Web Open in Cursor 

Co-authored-by: tika <tika@users.noreply.github.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
mhd Ready Ready Preview, Comment Apr 16, 2026 1:39am

@shaynesidman shaynesidman merged commit 95a3fb5 into main Apr 19, 2026
2 checks passed
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.

3 participants