Skip to content

read data type list to ensure we don't update existing#442

Merged
adamkorynta merged 1 commit intomainfrom
bugfix/fix_data_type_insert
Apr 8, 2025
Merged

read data type list to ensure we don't update existing#442
adamkorynta merged 1 commit intomainfrom
bugfix/fix_data_type_insert

Conversation

@adamkorynta
Copy link
Contributor

Problem Description

Fixes #420 .

Solution

read data type list to ensure we don't update existing and use the proper DbKey

how you tested the change

Unit tested and tested manually through web gui.

Where the following done:

  • Tests. Check all that apply:
    • Unit tests created or modified that run during ant test.
    • Integration tests created or modified that run during integration testing
      (Formerly called regression tests.)
    • Test procedure descriptions for manual testing
  • Was relevant documentation updated?
  • Were relevant config element (e.g. XML data) updated as appropriate

@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 8, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@adamkorynta adamkorynta merged commit 2d1155f into main Apr 8, 2025
8 of 9 checks passed
@adamkorynta adamkorynta deleted the bugfix/fix_data_type_insert branch April 8, 2025 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Saving a config results in an error.

3 participants