Skip to content

NPA-7384: Update examples and spec to ensure provision.actor is depre… - #389

Merged
ellie-bound1-NHSD merged 7 commits into
masterfrom
bug/NPA-7384/make-provision-actor-not-mandatory
Sep 1, 2026
Merged

NPA-7384: Update examples and spec to ensure provision.actor is depre…#389
ellie-bound1-NHSD merged 7 commits into
masterfrom
bug/NPA-7384/make-provision-actor-not-mandatory

Conversation

@ellie-bound1-NHSD

Copy link
Copy Markdown
Contributor

…cated and not to be used

Pull Request

🧾 Ticket Link

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


📄 Description/Summary of Changes

  • Make sure provision.actor is labeled as deprecated and remove it from examples.

🧪 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 August 27, 2026 10:28

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.

Pull request overview

Updates the OpenAPI specification and example payloads to discourage use of the deprecated Consent.provision.actor field by explicitly marking it deprecated in the spec and removing it from published examples, steering consumers toward policyRule (legal basis) instead.

Changes:

  • Mark Provision.actor as deprecated: true in the OpenAPI schema and update its description accordingly.
  • Remove provision.actor blocks from many GET/POST example YAMLs.
  • Update the PATCH policy rule example’s description (and associated guidance via the example reference).

Reviewed changes

Copilot reviewed 24 out of 24 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
specification/validated-relationships-service-api.yaml Deprecates Provision.actor in the schema (and updates description text).
specification/examples/responses/GET_Consent/single-mother-child-relationship.yaml Removes provision.actor from a single-relationship response example.
specification/examples/responses/GET_Consent/single-mother-child-relationship-unknown-legal-basis.yaml Removes provision.actor from an “unknown legal basis” response example.
specification/examples/responses/GET_Consent/single-mother-child-relationship-include-performer.yaml Removes provision.actor from a response example that includes performer.
specification/examples/responses/GET_Consent/single-mother-child-relationship-include-performer-patient.yaml Removes provision.actor from a response example that includes performer and patient.
specification/examples/responses/GET_Consent/single-mother-child-relationship-include-patient.yaml Removes provision.actor from a response example that includes patient.
specification/examples/responses/GET_Consent/single-consenting-adult-relationship.yaml Removes provision.actor from a single consenting-adult response example.
specification/examples/responses/GET_Consent/single-consenting-adult-relationship-include-performer.yaml Removes provision.actor from a consenting-adult response example that includes performer.
specification/examples/responses/GET_Consent/single-consenting-adult-relationship-include-performer-patient.yaml Removes provision.actor from a consenting-adult response example that includes performer and patient.
specification/examples/responses/GET_Consent/single-consenting-adult-relationship-include-patient.yaml Removes provision.actor from a consenting-adult response example that includes patient.
specification/examples/responses/GET_Consent/multiple-relationships.yaml Removes provision.actor from multiple-relationship response example entries.
specification/examples/responses/GET_Consent/multiple-relationships-single-patient.yaml Removes provision.actor from multiple-relationship (single patient) response example entries.
specification/examples/responses/GET_Consent/multiple-relationships-single-patient-include-performer.yaml Removes provision.actor from multiple-relationship (single patient) response example entries including performer.
specification/examples/responses/GET_Consent/multiple-relationships-single-patient-include-performer-patient.yaml Removes provision.actor from multiple-relationship (single patient) response example entries including performer and patient.
specification/examples/responses/GET_Consent/multiple-relationships-single-patient-include-patient.yaml Removes provision.actor from part of the multiple-relationship (single patient, include patient) response example.
specification/examples/responses/GET_Consent/multiple-relationships-include-performer.yaml Removes provision.actor from multiple-relationship response example entries including performer.
specification/examples/responses/GET_Consent/multiple-relationships-include-performer-patient.yaml Removes provision.actor from multiple-relationship response example entries including performer and patient.
specification/examples/responses/GET_Consent/multiple-relationships-include-patient.yaml Removes provision.actor from multiple-relationship response example entries including patient.
specification/examples/responses/GET_Consent/filtered-relationships-status-proposed-active.yaml Removes provision.actor from filtered-relationships response example entries.
specification/examples/responses/GET_Consent/filtered-relationships-status-inactive.yaml Removes provision.actor from the first inactive-status response example entry.
specification/examples/responses/GET_Consent/filtered-relationships-status-active-include-details.yaml Removes provision.actor blocks (including role/reference) from the include-details example.
specification/examples/requests/POST_Consent/parent_child_proxy_creation.yaml Removes actor from a POST consent creation request example.
specification/examples/requests/POST_Consent/adult_to_adult_proxy_creation.yaml Removes actor from a POST consent creation request example.
specification/examples/requests/PATCH_Consent/replace_policy_rule.yaml Updates the patch example description for changing the legal basis (policyRule).

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

Comment thread specification/validated-relationships-service-api.yaml
Comment on lines 2 to 6
summary: Replace policy rule
description: payload used to add a new policy rule to the consent provision
description: payload used to add a new legal basis to the proxy role
value:
- op: add
path: /policyRule
@ellie-bound1-NHSD
ellie-bound1-NHSD force-pushed the bug/NPA-7384/make-provision-actor-not-mandatory branch from e160c61 to a131a4b Compare August 28, 2026 15:04
@ellie-bound1-NHSD
ellie-bound1-NHSD force-pushed the bug/NPA-7384/make-provision-actor-not-mandatory branch from a131a4b to 89efb9c Compare September 1, 2026 07:57

@davesmallnhs davesmallnhs 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.

Yup

@sonarqubecloud

sonarqubecloud Bot commented Sep 1, 2026

Copy link
Copy Markdown

@ellie-bound1-NHSD
ellie-bound1-NHSD merged commit 8b44c81 into master Sep 1, 2026
18 of 19 checks passed
@ellie-bound1-NHSD
ellie-bound1-NHSD deleted the bug/NPA-7384/make-provision-actor-not-mandatory branch September 1, 2026 11:55
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