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

Add return requirements models to project #1071

Merged
merged 18 commits into from
Jun 6, 2024

Conversation

Cruikshanks
Copy link
Member

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

So far, the new returns requirements functionality supports setting up a new return version and its requirements. But the data is held only in a temporary session.

Alongside this, we've been making changes to the legacy return requirements tables so they can support the new properties we will record in our version of the returns requirement setup journey.

We now want to create new return requirements by copying an existing one, persist those that have been setup during the journey, and in the future, generate return logs using this information.

All this will require the tables to be represented as Objection.js models. This change adds the models, the views, and all the supporting elements in preparation for this work.

@Cruikshanks Cruikshanks added the enhancement New feature or request label Jun 4, 2024
@Cruikshanks Cruikshanks self-assigned this Jun 4, 2024
@Cruikshanks Cruikshanks force-pushed the add-return-requirement-models branch from 4d17bbe to 6712e1b Compare June 5, 2024 21:34
https://eaflood.atlassian.net/browse/WATER-4467

So far, the new returns requirements functionality supports setting up a new return version and its requirements. But the data is held only in a temporary session.

Alongside this, we've been making changes to the legacy return requirements tables so they can support the new properties we will record in our version of the returns requirement setup journey.

We now want to create new return requirements by copying an existing one, persist those that have been setup during the journey and in the future generate return logs using this information.

All this will need the tables to be represented as [Objection.js models](https://vincit.github.io/objection.js/). This change adds the models, the views and all the supporting elements in preparation for this work.
@Cruikshanks Cruikshanks force-pushed the add-return-requirement-models branch from 6712e1b to 27637ad Compare June 5, 2024 22:12
@Cruikshanks Cruikshanks marked this pull request as ready for review June 6, 2024 19:35
@Cruikshanks Cruikshanks merged commit 496e777 into main Jun 6, 2024
6 checks passed
@Cruikshanks Cruikshanks deleted the add-return-requirement-models branch June 6, 2024 19:35
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.

1 participant