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

Fix: 1453 No subsidiary workplace message in Summary panel #6286

Merged
merged 6 commits into from
Jul 25, 2024

Conversation

duncanc19
Copy link
Collaborator

@duncanc19 duncanc19 commented Jul 1, 2024

Work done

  • Ensure that number of child workplaces is always returned from missingCqcProviderLocations by moving setting outside of check for location ID

This resolves an issue where the message for no subsidiary workplaces was being displayed for non CQC parent workplaces when they did have subsidiaries

Tests

Does this PR include tests for the changes introduced?

  • Yes
  • No, I found it difficult to test
  • No, they are not required for this change

@duncanc19 duncanc19 changed the title Fix: 1453 Summary panel Fix: 1453 No subsidiary workplace message in Summary panel Jul 24, 2024
@duncanc19 duncanc19 merged commit 030e948 into main Jul 25, 2024
5 checks passed
@duncanc19 duncanc19 deleted the fix/1453-summary-panel-other-workplaces-message branch July 25, 2024 09:08
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