Skip to content

Feat: Student show page - #114

Open
sean-dickinson wants to merge 10 commits into
mainfrom
student-show-page
Open

Feat: Student show page#114
sean-dickinson wants to merge 10 commits into
mainfrom
student-show-page

Conversation

@sean-dickinson

Copy link
Copy Markdown
Collaborator

Resolves #100

Description

  • Updates the admin routing to use an explicit school show page with tabs for each of the different school views. Each tab is a lazy turbo frame that just renders the associated nested resource's index view.
  • The goal here is to simplify the routing for these nested resources while maintaining the expected conventions.

Screenshots

Screenshot 2026-08-29 at 15-45-17 Example School Screenshot 2026-08-29 at 15-45-26 Example School Screenshot 2026-08-29 at 15-45-52 Example School

@jonknapp jonknapp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with nit comments

Comment thread app/controllers/classrooms_controller.rb Outdated
Comment thread app/controllers/teachers_controller.rb Outdated
Comment thread app/views/schools/index.html.erb
Comment thread app/views/teachers/index.html.erb Outdated
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.

Update School Show Page

2 participants