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

Saving input from return requirements in session #939

Merged
merged 39 commits into from
Apr 30, 2024

Conversation

rvsiyad
Copy link
Contributor

@rvsiyad rvsiyad commented Apr 25, 2024

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

During the initial development of the return requirements pages, user input was not being saved in session. Later tickets and pages have this functionality in place. The session data is needed for the Check Your Answers page at the end of the journey.

This PR is focused on saving user input into the session where it is missing.

During the initial development of the return requirements pages, user input
was not being saved in session. Later tickets and pages have this functionality in place.
The session data is needed for the Check Your Answers page at the end of the journey.

This PR is focused on saving user input into the session where it is missing.
@rvsiyad rvsiyad added enhancement New feature or request housekeeping Refactoring, tidying up or other work which supports the project labels Apr 25, 2024
@rvsiyad rvsiyad self-assigned this Apr 25, 2024
rvsiyad added 26 commits April 25, 2024 13:21
@rvsiyad rvsiyad requested a review from Cruikshanks April 29, 2024 15:17
@rvsiyad rvsiyad marked this pull request as ready for review April 29, 2024 15:17
Copy link
Member

@Cruikshanks Cruikshanks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I hadn't realised that we'd made changes to Site Description to support replaying invalid entries in this PR.

That being the case, if we are making the changes we should look to copy the pattern we used for abstraction period. Shout if that is not clear and we can quickly pair and get this change completed!

@rvsiyad
Copy link
Contributor Author

rvsiyad commented Apr 30, 2024

I hadn't realised that we'd made changes to Site Description to support replaying invalid entries in this PR.

That being the case, if we are making the changes we should look to copy the pattern we used for abstraction period. Shout if that is not clear and we can quickly pair and get this change completed!

I added it in a couple days ago but I have now changed it to copy the new pattern created!

@rvsiyad rvsiyad requested a review from Cruikshanks April 30, 2024 10:22
@rvsiyad rvsiyad merged commit 3dcb83c into main Apr 30, 2024
6 checks passed
@rvsiyad rvsiyad deleted the return-requirements-saving-to-session branch April 30, 2024 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request housekeeping Refactoring, tidying up or other work which supports the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants