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 approve new return req. to use new points #1315

Merged
merged 3 commits into from
Sep 8, 2024

Conversation

Cruikshanks
Copy link
Member

https://eaflood.atlassian.net/browse/WATER-4600
https://eaflood.atlassian.net/browse/WATER-4645

We've been extending and amending the import of return versions from NALD to WRLS as part of our work to switch from NALD to WRLS to manage them.

In Update Return req set up journey to use new points we updated the return requirement setup journey to use the new water.licence_version_purpose_points table we've created and now populate thanks to a change to the import.

This change updates the logic behind the 'Approve' step in the journey to use the new points rather than those pulled from the permit.licence record.

https://eaflood.atlassian.net/browse/WATER-4600
https://eaflood.atlassian.net/browse/WATER-4645

We've been extending and amending the import of return versions from NALD to WRLS as part of our work to switch from NALD to WRLS to manage them.

In [Update Return req set up journey to use new points](#1301) we updated the return requirement setup journey to use the new `water.licence_version_purpose_points` table we've created and now populate thanks to [a change to the import](#1301).

This change updates the logic behind the 'Approve' step in the journey to use the new points rather than those pulled from the `permit.licence` record.
@Cruikshanks Cruikshanks added the housekeeping Refactoring, tidying up or other work which supports the project label Sep 8, 2024
@Cruikshanks Cruikshanks self-assigned this Sep 8, 2024
@Cruikshanks Cruikshanks marked this pull request as ready for review September 8, 2024 21:22
@Cruikshanks Cruikshanks merged commit 4069e7a into main Sep 8, 2024
6 checks passed
@Cruikshanks Cruikshanks deleted the update-approve-ret-requirements-for-new-points branch September 8, 2024 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Refactoring, tidying up or other work which supports the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant