Update validation msg for ret. req. copy existing #1112
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
https://eaflood.atlassian.net/browse/WATER-4283 > Part of the work to replace NALD for handling return requirements In [Complete Select existing return requirement page](#895) we added the controls and logic to enable a user to copy the return requirements from an existing return version into the session. The H1 title we used though was though to be wrong. So, we [Correct copy existing return req. page title](#1092). When we did that, we also updated the validation message as all our current messages just replay the H1 title. Again however, this was thought to be incorrect. Our UAT team have decided the message we should show when an option is not selected is **Select a previous requirements for returns**. This is that change.
Cruikshanks
added
the
housekeeping
Cruikshanks
requested review from
Jozzey,
jonathangoulding,
Beckyrose200 and
rvsiyad
Cruikshanks
added a commit
that referenced
this pull request
Jun 19, 2024
https://eaflood.atlassian.net/browse/WATER-4283 In [Update validation msg for ret. req. copy existing](#1112) we updated the validation message if a user doesn't select an option in the copy existing return requirements page to "Select a previous requirements for returns". Our design team however were originally going to use "Select a return version". Having seen the updated text it in the flesh, they've decided to go with their original plan. This change sets the validation message to be "Select a return version".
Cruikshanks
added a commit
that referenced
this pull request
Jun 19, 2024
https://eaflood.atlassian.net/browse/WATER-4283 In [Update validation msg for ret. req. copy existing](#1112) we updated the validation message if a user doesn't select an option in the copy existing return requirements page to "Select a previous requirements for returns". Our design team, however, was originally going to use "Select a return version". Having seen the updated text in 'the flesh', they've decided to go with their original plan. This change sets the validation message to "Select a return version".
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://eaflood.atlassian.net/browse/WATER-4283
In Complete Select existing return requirement page we added the controls and logic to enable a user to copy the return requirements from an existing return version into the session.
The H1 title we used though was though to be wrong. So, we Correct copy existing return req. page title. When we did that, we also updated the validation message as all our current messages just replay the H1 title.
Again, however, this was thought to be incorrect. Our UAT team has decided the message we should show when an option is not selected is Select a previous requirements for returns.
This is that change.