-
Notifications
You must be signed in to change notification settings - Fork 1
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
PRMP-744 PDS Error 20-A NACK #135
Conversation
… assumed INBOUND_COMPLETE status for outbound tests
@@ -14,6 +15,6 @@ export const getPdsOdsCode = async nhsNumber => { | |||
return response.data.data.odsCode; | |||
}) | |||
.catch(error => { | |||
throw new GetPdsCodeError(error); | |||
throw new GetPdsCodeError(error, AcknowledgementErrorCode.ERROR_CODE_20_A); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't the 20-A
implicit?
Report for environment: prodTerraform Format and Style 🖌
|
Report for environment: devTerraform Format and Style 🖌
|
Report for environment: testTerraform Format and Style 🖌
|
Report for environment: pre-prodTerraform Format and Style 🖌
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, happy with these changes after a couple of test failure tweaks.
This isn't one that we can test, wo will have to rely on the code testing.
No description provided.