-
Notifications
You must be signed in to change notification settings - Fork 10
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
DTSPB-4395 added new template id to caveat #2944
DTSPB-4395 added new template id to caveat #2944
Conversation
CCD diff reportNo change |
caseStoppedCaveat: "1fde1860-b827-45e7-b430-1406f74beb93" | ||
caseStoppedCaveat: "4aec6fa3-7e76-406a-9953-2f12d5d29f81" |
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.
- old https://www.notifications.service.gov.uk/services/4809d6a8-a10b-405d-88eb-1fee9b3042e3/templates/1fde1860-b827-45e7-b430-1406f74beb93
- new https://www.notifications.service.gov.uk/services/4809d6a8-a10b-405d-88eb-1fee9b3042e3/templates/4aec6fa3-7e76-406a-9953-2f12d5d29f81
new appears to only have changed the text in english, and the modified text doesn't match what is in the ticket
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.
resolved
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.
there are some other weird handling parts of these messages, but i think we can handle those outside the scope of the PR(?) - i have added a comment in the ticket to flag them and we can follow up through that since we can update the template without a PR
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.
Thanks Tom good catch .
CCD diff reportNo change |
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.
the alternative would be to directly update the existing template and make no change, but i think this is probably the better solution as we can track and control when that change goes live
CCD diff reportNo change |
CCD diff reportNo change |
@@ -286,7 +286,7 @@ notifications: | |||
applicationReceivedPaperFormCaseworker: "22c81233-a213-4520-b6bc-896fe8f6988a" | |||
documentReceived: "83f93f41-58a1-44f2-a9c4-d20d545b7528" | |||
caseStopped: "a767a0bd-4acc-413e-b510-5a71d24daf85" | |||
caseStoppedCaveat: "937291a0-c0ae-40a4-a8cf-46a3452faff0" | |||
caseStoppedCaveat: "d5084257-f475-415a-b7d0-2427b5c580b1" |
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.
this should use welsh version template
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.
resolved
CCD diff reportNo change |
CCD diff reportNo change |
private static final String REGISTRY_NO = "0113 389 6133"; | ||
private static final String REGISTRY_NO = " 0300 303 0648"; |
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.
can we update the test(s) where this was failing to improve their failure messaging in case we have the same issue in the future? (given the error message was org.opentest4j.AssertionFailedError: expected: <true> but was: <false>
this doesn't actually provide any useful information on why it is failing.)
CCD diff reportNo change |
Plan Result (aat)
|
Plan Result (prod)
|
CCD diff reportNo change |
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/DTSPB-4395
Change description
Does this PR introduce a breaking change? (check one with "x")