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 bill run setup journey to use new session #983

Merged
merged 4 commits into from
May 6, 2024

Conversation

Cruikshanks
Copy link
Member

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

We created our SessionModel and its backing table out of the need to be able to store data temporarily, for example, when a user is following a journey to create a new return requirement.

After using it in our bill run setup and return requirement journeys we spotted there were some improvements we could make. So, we Enhanced the SessionModel to expose and update data.

The next step is to update the existing journeys to take advantage of these enhancements.

In this change, we update the bill run setup journey. We'll update the return requirements journey in the next change.

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

We created our `SessionModel` and its backing table out of the need to be able to store data temporarily, for example, when a user is following a journey to create a new return requirement.

After using it in our bill run setup and return requirement journeys we spotted there were some improvements we could make. So, we [Enhanced the SessionModel to expose and update data](#981).

The next step is to update the existing journeys to take advantage of these enhancements.

In this change we update the bill run setup journey. We'll update the return requirements journey in the next change.
@Cruikshanks Cruikshanks added the housekeeping Refactoring, tidying up or other work which supports the project label May 6, 2024
@Cruikshanks Cruikshanks self-assigned this May 6, 2024
As the note explains. The enhancements mean we cannot just instantiate an instance and pass it in like we'd just fetched it!
@Cruikshanks Cruikshanks marked this pull request as ready for review May 6, 2024 09:43
@Cruikshanks Cruikshanks merged commit 99b4393 into main May 6, 2024
6 checks passed
@Cruikshanks Cruikshanks deleted the update-session-based-journies branch May 6, 2024 09:44
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