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.

Sign up for GitHub

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

Jump to bottom

Update Breadcrumbs to accommodate users with multiple facility_ids #113

Open
Tracked by #120
michaelkohn opened this issue Apr 29, 2024 · 4 comments · May be fixed by medic/cht-core#9093
Open
Tracked by #120

Update Breadcrumbs to accommodate users with multiple facility_ids #113

michaelkohn opened this issue Apr 29, 2024 · 4 comments · May be fixed by medic/cht-core#9093
Assignees
@latin-panda
Labels
Supervisor

Comments

@michaelkohn
Copy link
Member

#6543 allows users to have more than one facility_id.

Breadcrumbs are shown on the Messages, Tasks, and Reports pages. In medic/cht-core#5697 we removed a level for the user's home place, but now that users can have data from multiple places, we need to show them for those users.

If the user has more than one facility_id, show all breadcrumbs in the

  • Messages Page | List View
  • Tasks Page | List View
  • Reports Page | List View

No changes needed on the Detail View of those pages.

See full list of changes here
@michaelkohn michaelkohn mentioned this issue Apr 30, 2024
Open
6 tasks
@latin-panda latin-panda self-assigned this May 2, 2024
@latin-panda latin-panda mentioned this issue May 3, 2024
Closed
@latin-panda latin-panda linked a pull request May 3, 2024 that will close this issue
feat(#6543): add back breadcrumbs for users with many facilities medic/cht-core#9093
Open
5 tasks
@latin-panda
Copy link
Collaborator

Feature ready, I need to work more on the test coverage in the code next week (unit tests, e2e tests).
Videos here

@latin-panda
Copy link
Collaborator

latin-panda commented May 9, 2024 •
edited

This is complete and approved by reviewers, but I would like to wait until Diana finishes her part to not get in the way in some way.

@latin-panda
Copy link
Collaborator

This has been rebased, and the tests are working. No major changes are expected, so it's ready. I'm just waiting for Diana's 6543-multi-facility.

@latin-panda
Copy link
Collaborator

latin-panda commented May 22, 2024 •
edited

@Benmuiruri might need to take over and get it to the finish line

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@latin-panda latin-panda

Labels
Supervisor
Projects
Care Teams
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(#6543): add back breadcrumbs for users with many facilities medic/cht-core
2 participants
@michaelkohn @latin-panda