Skip to content
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

Fix back link for copy existing in rtn vers setup #1494

Merged
merged 6 commits into from
Nov 18, 2024

Conversation

Cruikshanks
Copy link
Member

https://eaflood.atlassian.net/browse/WATER-4283

We noticed when prepping for a discussion about return version setup that going back from the 'Use previous requirements for returns' page in the return version setup journey results in a 404.

Basically, when we carried out a refactor to Rename return-requirements route return-versions, we didn't spot that the backlink had been built differently than all the other pages in the journey.

So, it is still trying to go back to the defunct route. This change fixes it.

https://eaflood.atlassian.net/browse/WATER-4283

Spotted when prepping for a discussion about return versions setup that going back from the 'Use previous requirements for returns' page in the return version setup journey results in a 404.

Basically, when we carried out a refactor to [Rename return-requirements route return-versions](#1431) we didn't spot that the back link had been built differently than all the other pages in the journey.

So, it is still trying to go back to the defunct route. This changes fixes it.
@Cruikshanks Cruikshanks added the bug Something isn't working label Nov 15, 2024
@Cruikshanks Cruikshanks self-assigned this Nov 15, 2024
@Cruikshanks Cruikshanks merged commit f431167 into main Nov 18, 2024
6 checks passed
@Cruikshanks Cruikshanks deleted the fix-go-back-from-rtn-version-setup-copy branch November 18, 2024 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants