search

medic / care-teams

For Product Management
0 stars 0 forks source link

Disable Aggregate Targets for users with multiple facility_ids #114

Closed michaelkohn closed 3 weeks ago

michaelkohn commented 2 months ago

#6543 allows users to have more than one facility_id.

It doesn't make sense to aggregate targets data across mutliple facility_ids so we'd need a significant UI change to accommodate this. For the MVP, we've chosen just to disable aggregate targets for users with multiple facility_ids.

See full list of changes here

latin-panda commented 1 month ago

PR ready for review waiting for feedback.

latin-panda commented 1 month ago

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 commented 1 month ago

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 commented 1 month ago

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