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

Return Logs setup - Period used #1660

Merged
merged 20 commits into from
Feb 5, 2025
Merged

Return Logs setup - Period used #1660

merged 20 commits into from
Feb 5, 2025

Conversation

Beckyrose200
Copy link
Contributor

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

As part of the work to migrate the return logs setup journey, this PR is focused on adding the 'What period was used for this volume?' page.

NOTE: This work is only for a due return. The pages will be updated in a separate PR to handle the editing of existing return logs.

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

As part of the work to migrate the return logs setup journey, this PR is focused on adding the 'What period was used for this volume?' page.

NOTE: This work is only for a due return. The pages will be updated in a separate PR to handle the editing of existing return logs.
@Beckyrose200 Beckyrose200 added the enhancement New feature or request label Jan 27, 2025
@Beckyrose200 Beckyrose200 self-assigned this Jan 27, 2025
@Beckyrose200
Copy link
Contributor Author

Please ignore SonarClouds complaint about code duplication. Its because the Submit Service looks the same to every other Submit Service and it doesn't like it :)

@Beckyrose200 Beckyrose200 marked this pull request as ready for review February 5, 2025 14:24
@Beckyrose200 Beckyrose200 requested review from Cruikshanks, Jozzey and rvsiyad and removed request for Cruikshanks and Jozzey February 5, 2025 14:24
@Beckyrose200 Beckyrose200 requested a review from StuAA78 February 5, 2025 14:29
@Beckyrose200 Beckyrose200 merged commit bc298c4 into main Feb 5, 2025
6 of 7 checks passed
@Beckyrose200 Beckyrose200 deleted the return-logs-period-used branch February 5, 2025 17:48
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.

2 participants