Skip to content

NPA-5140: GET /Consent Invalid Status Wrong Error Code#225

Closed
LeoKHoward wants to merge 3 commits into
masterfrom
task/NPA-5140/update-get-consent-invalid-status-code
Closed

NPA-5140: GET /Consent Invalid Status Wrong Error Code#225
LeoKHoward wants to merge 3 commits into
masterfrom
task/NPA-5140/update-get-consent-invalid-status-code

Conversation

@LeoKHoward

@LeoKHoward LeoKHoward commented Jun 19, 2025

Copy link
Copy Markdown
Contributor

Pull Request

Ticket Link

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

Description/Change Summary

Updated INVALID_VALUE code and diagnostics message to match INVALID_PARAMETER one

How to test?

Tested after merge

Review Checklist

ℹ️ This section is to be filled in by the reviewer.

  • I have reviewed the changes in this PR and they fill all or part of the acceptance criteria of the ticket, and the code is in a mergeable state.
  • If there were infrastructure, operational, or build changes, I have made sure there is sufficient evidence that the changes will work.
  • I have ensured the changelog has been updated by the submitter, if necessary.

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.

@github-actions

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket:

NPA-5140

@LeoKHoward LeoKHoward self-assigned this Jun 19, 2025
@LeoKHoward LeoKHoward added bug Something isn't working postman Changes to the postman collection labels Jun 19, 2025
Comment thread postman/Validate Relationship Service Sandbox.postman_collection.json Outdated
Comment thread specification/examples/responses/errors/invalid-include-parameter.yaml Outdated
" }",
" ],",
" \"diagnostics\": \"Invalid request with error - _include parameter is invalid.\"",
" \"diagnostics\": \"Invalid request with error - status parameter is invalid.\"",

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.

Neither before or new change seems to be correct looking at the request
"raw": "{{api_base_url}}/Consent/74eed847-ca25-4e76-8cf2-f2c2d7842a7a?_unknown=true

@github-actions

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket:

NPA-5140

@github-actions

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital NPA JIRA Project. Here's a handy link to the ticket:

NPA-5140

@LeoKHoward LeoKHoward marked this pull request as draft June 23, 2025 08:57
@LeoKHoward LeoKHoward closed this Jul 10, 2025
@LeoKHoward LeoKHoward deleted the task/NPA-5140/update-get-consent-invalid-status-code branch July 10, 2025 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working postman Changes to the postman collection

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants