Skip to content

Feat: Students CRUD#16

Open
sean-dickinson wants to merge 9 commits intoadmin-schoolsfrom
students-crud
Open

Feat: Students CRUD#16
sean-dickinson wants to merge 9 commits intoadmin-schoolsfrom
students-crud

Conversation

@sean-dickinson
Copy link
Copy Markdown
Collaborator

@sean-dickinson sean-dickinson commented Apr 23, 2026

What

  • Adds Student CRUD screens
  • Removes useless local CI step for testing seeds (I don't run seeds in the test environment)

Why

  • Clearly there are already opportunities for drying things up regarding the forms, breadcrumbs, styling etc. For now I'm leaving everything as copy/paste until there's a clear enough pattern that's worth abstracting.
  • Note that I chose keeping the students resource shallow nested rather than fully nested as I think this is a better call for the long term as otherwise it can get awkward to use the route helpers.

@sean-dickinson sean-dickinson changed the base branch from main to admin-schools April 23, 2026 00:31
@sean-dickinson sean-dickinson marked this pull request as ready for review April 26, 2026 17:28
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