Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update Coach routing behavior #10650

Closed
thanksameeelian opened this issue May 6, 2023 · 0 comments · Fixed by #11224
Closed

update Coach routing behavior #10650

thanksameeelian opened this issue May 6, 2023 · 0 comments · Fixed by #11224
Assignees
Labels
APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) bug Behavior is wrong or broken P0 - critical Priority: Release blocker or regression TAG: ux update Improved user-facing feature

Comments

@thanksameeelian
Copy link
Contributor

thanksameeelian commented May 6, 2023

Observed behavior

when a user is a multi-facility admin & selects a Coach subtopic from sidenav, they will be redirected to an "All Facilities" page with links leading to each facility's "Class home" page (or "All classes" page if facility has multiple classes, the links on which resolve to "Class home" for each class), rather than the subtopic the user initially selected.

Proposed behavior

mirror the routing updates introduced in #10550:

  • if a multi-facility user selects Coach subtopic, they should be redirected to an "All Facilities" page populated with links to each facility's page for that subtopic (e.g. if a multi-facility user has a facility with multiple classes, clicking on Plan in the sidenav will lead you to "All Facilities", and clicking on the facility will lead to the "All classes" page, where the links for each class resolve directly to that class's Plan when selected - in facility tab navigation bug #10550 similar routing is done with subtopic names passed as route params that are taken in as props by the "All facilities" route)
  • adjust backlink to retain information about the subtopic being navigated away from & use that prop to construct subtopic-specific links on "All Facilities"/"All Classes" page (PR addressing Missing 'All classes' link on device with multiple facilities #10673 will add back links when needed into all main Coach subtopic pages, but will not introduce passing subtopic-specific information during navigation)

User-facing consequences

Users must select a subtopic in order to navigate to Coach plugin but, in the case of a multi-facility user, once they select a link on the "All Facilities" page, they will always end up getting directed to "Class home" regardless of their initial selection

@thanksameeelian thanksameeelian added APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) TAG: ux update Improved user-facing feature labels May 6, 2023
@marcellamaki marcellamaki added bug Behavior is wrong or broken P0 - critical Priority: Release blocker or regression labels May 23, 2023
@thanksameeelian thanksameeelian changed the title update Coach routing behavior & add backlinks to "All Facilities" page in all subtopics update Coach routing behavior May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) bug Behavior is wrong or broken P0 - critical Priority: Release blocker or regression TAG: ux update Improved user-facing feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants