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 param_1_label & param_2_label to lvpct view #1569

Conversation

rvsiyad
Copy link
Contributor

@rvsiyad rvsiyad commented Dec 16, 2024

DEFRA/water-abstraction-service#2663

As part of the migration of legacy pages into the new system, we identified missing information in the licence_version_purpose_condition_types table. Specifically, the param1 and param2 values in the licence_version_purpose_conditions table are dynamically associated with row titles derived from the code and subcode in the licence_version_purpose_condition_types table. To correctly reference these row titles for each parameter, we added the static values to the licence_version_purpose_condition_types table. We now need to add the param_1_label and param_2_label to the licence_version_purpose_condition_types view.

This PR introduces two new columns,param_1_label and param_2_label to the licence_version_purpose_condition_types view.

DEFRA/water-abstraction-service#2663

As part of the migration of legacy pages into the new system, we identified missing information in the licence_version_purpose_condition_types table.
Specifically, the param1 and param2 values in the licence_version_purpose_conditions table are dynamically associated with row titles derived from the
code and subcode in the licence_version_purpose_condition_types table. To correctly reference these row titles for each parameter, we added the
static values to the licence_version_purpose_condition_types table. We now need to add the `param_1_label` and `param_2_label` to the licence_version_purpose_condition_types
view.

This PR introduces two new columns,`param_1_label` and `param_2_label` to the licence_version_purpose_condition_types view.
@rvsiyad rvsiyad added the housekeeping Refactoring, tidying up or other work which supports the project label Dec 16, 2024
@rvsiyad rvsiyad self-assigned this Dec 16, 2024
@rvsiyad rvsiyad marked this pull request as ready for review December 17, 2024 17:14
Copy link
Contributor

@Jozzey Jozzey left a comment

Choose a reason for hiding this comment

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

@rvsiyad rvsiyad merged commit c2468ba into main Dec 18, 2024
7 checks passed
@rvsiyad rvsiyad deleted the add-param1label-and-param2label-columns-to-licence-version-purpose-condition-types-view branch December 18, 2024 10:11
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.

2 participants