updates peers list - #9
Conversation
tomjemmett
commented
Jul 22, 2026
tomjemmett
commented
- uses peers list from model health system
- for specialist providers that aren't in that list have been added in 3 groups (womens and childrens, orthopaedic and neurology, other specialist)
- the remaining two providers (UHB, Croydon) used the older peers
- uses peers list from model health system - for specialist providers that aren't in that list have been added in 3 groups (womens and childrens, orthopaedic and neurology, other specialist) - the remaining two providers (UHB, Croydon) used the older peers
There was a problem hiding this comment.
Pull request overview
This PR updates how the app determines and displays provider peer groups by switching from an internal RDS peers dataset to a JSON peers mapping (aligned to the model health system), and adjusting server-side peer selection logic accordingly.
Changes:
- Replace
peers.Rdsloading withinst/peers.jsonloading viayyjsonr::read_json_file(). - Simplify peer selection to a direct
%in%filter onorg_idusing the selected provider and its peer list. - Adjust peer ID extraction sent to the front-end map handler.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| R/app_server.R | Loads peers from JSON and updates the peer-selection/filtering + peer ID extraction logic. |
| inst/peers.json | Adds the new peers mapping data used by the server for peer selection. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
I'm checking again with @AlisonJaneWright and dev board whether we can go ahead and make this change. I've sent her an excel file that shows, particularly for active schemes, how the incoming OMHS peers would change their list of peers. We should not merge this until Alison and dev board have signed off to give it the go ahead. |
