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

Move Return reqs. pages into setup folder #1429

Merged
merged 8 commits into from
Oct 20, 2024

Conversation

Cruikshanks
Copy link
Member

@Cruikshanks Cruikshanks commented Oct 20, 2024

DEFRA/water-abstraction-team#126

We are on the cusp of our work to migrate the management of return versions (requirements) from NALD to WRLS going 'live'.

This is a significant feature that has seen many iterations since we started. So, before it gets used in earnest, we want to ensure it is 'ship shape and Bristol fashion'!

A crucial part of the tidy-up is moving all modules related to setting up a return version into a /setup subfolder, using the same pattern we used for setting up a bill run. We also move the routes to /return-requirements/setup/{sessionId.

This helps us distinguish pages used solely to 'set up' a new return requirement from those used to view and manage them.

DEFRA/water-abstraction-team#126

We are on the cusp of our work to migrate the management of return versions (requirements) from NALD to WRLS going 'live'.

This is a significant feature that has seen many iterations since we started. So, before it gets used in earnest, we want to ensure it is 'ship shape and Bristol fashion'!

A key part of the tidy-up is moving all modules related to setting up a return version into a `/setup` subfolder, using the same pattern we used for setting up a bill run.

Doing this helps us distinguish pages used solely for the purpose of 'setting up' a new return requirement, from those used to view and manage them.
@Cruikshanks Cruikshanks added the housekeeping Refactoring, tidying up or other work which supports the project label Oct 20, 2024
@Cruikshanks Cruikshanks self-assigned this Oct 20, 2024
@Cruikshanks Cruikshanks marked this pull request as ready for review October 20, 2024 20:47
@Cruikshanks Cruikshanks merged commit 0e88d29 into main Oct 20, 2024
5 of 6 checks passed
@Cruikshanks Cruikshanks deleted the move-rtn-req-into-setup-folder branch October 20, 2024 20:48
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