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

Data model supports multiple providers and default provider #8271

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mehansen
Copy link
Collaborator

@mehansen mehansen commented Nov 6, 2024

BACKEND PULL REQUEST

Related Issue

first of probably 4 PRs to implement #7982

Changes Proposed

  • change ordering provider column to be default ordering provider
  • add concept of multiple providers to a facility
  • when a new facility is created, the ordering provider is set as the default and added to the list of providers

Additional Information

next PR will handle:

  • support adding, removing providers and updating the default provider, as well as constraints around facilities having at least 1 provider
  • update org initialization logic to fix how we're creating the default provider and facilities (e.g. see number of identical providers in your local DB)
  • potentially making the default_ordering_provider_id column nullable

Testing

DB migration should populate facility_providers table

test that existing provider related behavior is not impacted:

  • create facility with a provider
  • update provider
  • delete a facility (provider is not marked as deleted)

Copy link

sonarcloud bot commented Nov 15, 2024

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.

2 participants