Skip to content

NPA-7488: Fix misalignment of misuse status reason - #394

Merged
ehallam merged 1 commit into
masterfrom
task/NPA-7488/fix-misalignment-of-misuse-status-reason-bug
Sep 17, 2026
Merged

ehallam merged 1 commit into
masterfrom
task/NPA-7488/fix-misalignment-of-misuse-status-reason-bug

Conversation

@ehallam

@ehallam ehallam commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Pull Request

🧾 Ticket Link

https://nhsd-jira.digital.nhs.uk/browse/NPA-7488


📄 Description/Summary of Changes

  • Changed ROLE_DEACTIVATED_MISUSE_OF_SYSTEM to be ROLE_DEACTIVATED_SYSTEM_MISUSE

🧪 Developer Testing Carried Out


📋 PR Principles

  • Keep PRs Small and Focused: Ensure the PR addresses a single task or feature to make it easier to review.
  • Multiple PRs for one Ticket: When splitting work into multiple PRs, clearly describe what this PR addresses and outline the remaining work to complete the ticket.
  • Ensure Tests Are Included: Add or update unit, integration, or end-to-end tests to cover the changes made.
  • Follow Coding Standards: Ensure the code adheres to the team's coding guidelines and best practices.
  • Resolve Comments Promptly: If you raise a comment, ensure you follow up and resolve it before approving the PR to maintain clarity and ensure comments are addressed.
  • Foster Learning: PR reviews are an opportunity to share knowledge, provide constructive feedback, and encourage a collaborative environment.

🏷️ Naming Conventions Reminder

Please ensure the following naming conventions are followed:

  • PR title follows the format: NPA-XXXX: <short-description>
  • Branch name follows the convention: <type>/NPA-XXXX/<short-description>
  • Commit messages follow the template: NPA-XXXX: <short-description>

Copilot AI lite review requested due to automatic review settings September 17, 2026 12:05
@sonarqubecloud

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

Bump the schema version to reflect the public contract change.

Pull request overview

Updates the OpenAPI contract to correct the misuse status-reason code and documentation.

Changes:

  • Renames the enum value to ROLE_DEACTIVATED_SYSTEM_MISUSE.
  • Updates its documented display mapping.
File summaries
File Description
specification/validated-relationships-service-api.yaml Corrects the status-reason enum and documentation.
Review details

Suppressed comments (1)

specification/validated-relationships-service-api.yaml:2498

  • This changes the public StatusReasonCodeableConcept enum, but info.version remains 1.28.0. The repository's release guidance requires updating the schema version for a new schema contract; without that, published documentation and generated clients cannot distinguish this corrected enum from the previous contract. Please bump the specification version as part of this change.
                  - ROLE_DEACTIVATED_SYSTEM_MISUSE
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ehallam
ehallam merged commit b7ec0c1 into master Sep 17, 2026
22 checks passed
@ehallam
ehallam deleted the task/NPA-7488/fix-misalignment-of-misuse-status-reason-bug branch September 17, 2026 14:39
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.

3 participants