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

Create model for purposes_uses table #335

Merged
merged 14 commits into from
Aug 4, 2023
Merged

Conversation

Jozzey
Copy link
Contributor

@Jozzey Jozzey commented Aug 3, 2023

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

Whilst working on ticket 4046 we have realised that we need a model for the purposes_uses table to be able to match the Returns to a charge purpose. The Return has a value in the metadata that matches to the legacy_id in the purposes_uses table.

I also noticed while working on this model that we hadn't used the correct naming conventions for the file names of the returns schema migrations that we had recently built. Also in the migration 20230802134153_create-returns-returns the constraints & defaults had been omitted so I also added these.

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

Whilst working on ticket 4046 we have realised that we need a model for the `purpose_uses` table to be able to match the Returns to a charge purpose. The Return has a value in the `metadata` that matches to the `legacy_id` in the `purpose_uses` table.
@Jozzey Jozzey added the enhancement New feature or request label Aug 3, 2023
@Jozzey Jozzey marked this pull request as ready for review August 3, 2023 18:08
@Jozzey Jozzey changed the title Create model for purpose_uses table Create model for purposes_uses table Aug 3, 2023
@Jozzey Jozzey merged commit 2e80abf into main Aug 4, 2023
@Jozzey Jozzey deleted the create-model-for-purpose_uses branch August 4, 2023 09:55
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.

3 participants