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

Refactor the existing purpose condition types to be consistent #1218

Merged

Conversation

jonathangoulding
Copy link
Collaborator

@jonathangoulding jonathangoulding commented Jul 31, 2024

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

As part of migrating the import service into the wabs system repo we are adding seeders to the tests which will add what we call reference data the test database.

This is currently running and adding other seeders in the test database.

This change aligns the licence version purpose condition type seeder. Mainly using a data file for us to access this static data in the tests and using the build value string function to build the values to insert into the database.

As part of migrating the import service into the wabs system repo we are adding seeders to the tests which will add what we call reference data the test database.

This is currently running and adding other seeders in the test database.

This change aligns the licence version purpose condition type seeder. Mainly using a data file for us to access this static data in the tests and using the build value string function to build the values to insert into the database.

https://eaflood.atlassian.net/browse/WATER-4535
@jonathangoulding jonathangoulding self-assigned this Jul 31, 2024
@jonathangoulding jonathangoulding marked this pull request as ready for review July 31, 2024 11:17
@jonathangoulding jonathangoulding merged commit 8c4d677 into main Jul 31, 2024
6 checks passed
@jonathangoulding jonathangoulding deleted the refactor-licence-version-purpose-contdition-type-seeder branch July 31, 2024 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants