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

Update use previous in ret. req setup with mod log #1282

Merged
merged 8 commits into from
Aug 21, 2024

Conversation

Cruikshanks
Copy link
Member

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

Part of the work to display a licence's history to users (mod log)

In Update view return version to include mod log info, we updated the view return version page to include details taken from a return version's mod log history.

Then, in Update return versions table with reasons in view, we updated the table of return requirements in the view licence setup tab to display the mod log reason for return versions without a mapping.

The final place we display return version reasons is the 'Use previous requirements for returns' page in the return version set-up journey.

This change updates the logic for return versions without a reason but that have a mod log with a reason: we use its description.

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

> Part of the work to display a licence's history to users (mod log)

In [Update view return version to include mod log info](#1261), we updated the view return version page to include details taken from a return version's mod log history.

Then in [Update return versions table with reasons in view](#1281) we updated the table of return requirements in the view licence setup tab to display the mod log reason for return versions without a mapping.

The final place we display return version reasons is the 'Use previous requirements for returns' page in the return version set up journey.

This change updates the logic so for return versions without a reason but that have a mod log with a reason, we use its description.
@Cruikshanks Cruikshanks added the enhancement New feature or request label Aug 21, 2024
@Cruikshanks Cruikshanks self-assigned this Aug 21, 2024
@Cruikshanks Cruikshanks marked this pull request as ready for review August 21, 2024 23:49
@Cruikshanks Cruikshanks merged commit 8638488 into main Aug 21, 2024
6 checks passed
@Cruikshanks Cruikshanks deleted the update-use-previous-with-mod-log branch August 21, 2024 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant