NPA-6154: Change the spec and Postman#299
Conversation
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
03db0ab to
c53e8c6
Compare
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
ellie-bound1-NHSD
left a comment
There was a problem hiding this comment.
Need to remove provision.actor.role from examples and in the postman as we are deprecating it. have added a few comments where to remove but applies too all places. Please remove 🙏
| value: '9000000019' | ||
| role: | ||
| - role: | ||
| coding: |
There was a problem hiding this comment.
Please remove the whole role object
So that we are not showing a deprecated property in our examples
so line 39 to 43 should be gone as that is what you have replaced with regulatory basis
| @@ -45,9 +45,9 @@ ConsentSingleAdultChildRelationshipUnknownLegalBasisBundle: | |||
| type: RelatedPerson | |||
| role: | |||
There was a problem hiding this comment.
Please remove role
| - code: parental-responsibility | ||
| display: Parental responsibility | ||
| system: https://fhir.nhs.uk/England/CodeSystem/England-NHSProxyLegalBasis | ||
| coding: |
There was a problem hiding this comment.
Please remove role
| - code: parental-responsibility | ||
| display: Parental responsibility | ||
| system: https://fhir.nhs.uk/England/CodeSystem/England-NHSProxyLegalBasis | ||
| coding: |
There was a problem hiding this comment.
Please remove role
| @@ -163,8 +170,8 @@ ConsentMultipleRelationshipsSinglePatientIncludePerformerPatientBundle: | |||
| value: '9000000021' | |||
| role: | |||
There was a problem hiding this comment.
Please remove role
| @@ -93,9 +93,10 @@ ConsentMultipleRelationshipsSinglePatientIncludePerformerPatientBundle: | |||
| value: '9000000020' | |||
| role: | |||
There was a problem hiding this comment.
Please remove role
| @@ -175,8 +187,8 @@ ConsentMultipleRelationshipsSinglePatientIncludePatientBundle: | |||
| value: '9000000022' | |||
| role: | |||
There was a problem hiding this comment.
Please remove role
| @@ -123,8 +129,8 @@ ConsentMultipleRelationshipsSinglePatientIncludePatientBundle: | |||
| value: '9000000021' | |||
| role: | |||
There was a problem hiding this comment.
Please remove role
| @@ -71,8 +71,8 @@ ConsentMultipleRelationshipsSinglePatientIncludePatientBundle: | |||
| value: '9000000020' | |||
| role: | |||
There was a problem hiding this comment.
Please remove role
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
| @@ -141,7 +147,7 @@ ConsentMultipleRelationshipsIncludePerformerBundle: | |||
| role: | |||
| identifier: | ||
| system: https://fhir.nhs.uk/Id/nhs-number | ||
| value: '9000000022' | ||
| role: |
| identifier: | ||
| system: https://fhir.nhs.uk/Id/nhs-number | ||
| value: '9000000022' | ||
| role: |
| identifier: | ||
| system: https://fhir.nhs.uk/Id/nhs-number | ||
| value: '9000000022' | ||
| role: |
| identifier: | ||
| system: https://fhir.nhs.uk/Id/nhs-number | ||
| value: '9000000017' | ||
| role: |
| system: "https://fhir.nhs.uk/Id/nhs-number" | ||
| value: "9000000017" | ||
| role: | ||
| - role: |
| @@ -25,10 +25,10 @@ ConsentAdultToAdultGuardianRequest: | |||
| start: "2026-02-11T14:30:00Z" | |||
| actor: | |||
| - role: | |||
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
ellie-bound1-NHSD
left a comment
There was a problem hiding this comment.
A few performers are added back into postman
There was a problem hiding this comment.
Would expect this to be different if following these docs -> https://nhsd-confluence.digital.nhs.uk/spaces/NPA/pages/874694621/How+to+update+Postman+Collections
There was a problem hiding this comment.
I reimported, updated to follow date format and exported back into the repo. So hopefully this is now correct?
| " }", | ||
| " },", | ||
| " \"dateTime\": \"2024-07-21T17:32:28Z\",", | ||
| " \"performer\": [", |
There was a problem hiding this comment.
why is performer back?
There was a problem hiding this comment.
Should now all be removed
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
…thub.com/NHSDigital/validated-relationships-service-api into task/NPA-6154/regulatory-basis-extension
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
|
This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket: NPA-6154 |
| system: "https://fhir.nhs.uk/Id/nhs-number" | ||
| value: "9000000017" | ||
| role: | ||
| - role: |
<!-- Add the Jira ticket link here --> https://nhsd-jira.digital.nhs.uk/browse/NPA-6154 --- <!-- Describe the changes made in this PR. Include the purpose/scope/impact of the changes --> - Update Postman - Update Spec - <!-- Add bullet points summarising key changes --> --- <!-- Describe what tests (automated/unit/manual etc.) have been done for the ticket. Include: --> <!-- - Any tests added/updated --> <!-- - Evidence that each acceptance criterion from the Jira ticket is met --> - <!-- Add bullet points for testing instructions --> - <!-- Add bullet points for testing instructions --> - <!-- Add bullet points for testing instructions --> --- <!-- Describe how to test the changes that have been made in the ticket. Include: --> <!-- - Testing environment details (e.g. sandbox/local setup) --> <!-- - Steps to verify the changes --> - [ ] <!-- Add bullet points for testing instructions --> - [ ] <!-- Add bullet points for testing instructions --> - [ ] <!-- Add bullet points for testing instructions --> --- <!-- Complete before submitting the PR --> - [x] PR title follows the format: `NPA-XXXX: <short-description>` - [x] Branch name follows the convention: `<type>/NPA-XXXX/<short-description>` - [x] Commit messages follow the template: `NPA-XXXX: <short-description>` - [ ] All acceptance criteria from the Jira ticket are addressed - [ ] Automated tests (unit/integration/API/infrastructure etc. tests) are added or updated - [ ] Assignees and appropriate labels (e.g. `terraform`, `documentation`) are added --- <!-- To be completed by the reviewer --> - [ ] Changes meet the acceptance criteria of the Jira ticket - [ ] Code is able to be merged (no conflicts and adheres to coding standards) - [ ] Sufficient test evidence is provided (manual and/or automated) - [ ] Infrastructure/operational/build changes are validated (if applicable) --- <!-- Actions to complete after merging --> After merging and deploying changes to the sandbox, Postman collection or spec examples please run the Run Postman collection workflow. This will run the tests within the collection to check that the sandbox is working as expected once deployed. --------- Co-authored-by: Leo Howard <38526889+LeoKHoward@users.noreply.github.com>
Pull Request
🧾 Ticket Link
https://nhsd-jira.digital.nhs.uk/browse/NPA-6154
📄 Description/Summary of Changes
🧪 Developer Testing Carried Out
🧪 Reviewer Testing Required
✅ Developer Checklist
NPA-XXXX: <short-description><type>/NPA-XXXX/<short-description>NPA-XXXX: <short-description>terraform,documentation) are added👀 Reviewer Checklist
🚀 Post-merge
After merging and deploying changes to the sandbox, Postman collection or spec examples please run the Run Postman
collection workflow.
This will run the tests within the collection to check that the sandbox is working as expected once deployed.