Skip to content

fix query param separator in ajax request#433

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

fix query param separator in ajax request#433
adamkorynta merged 1 commit intomainfrom
bugfix/delete_unit_conversion

Conversation

@adamkorynta
Copy link
Contributor

fixes deleting EU conversion delete

Problem Description

Cannot delete unit conversion.

Fixes #417 .

Solution

Fix ajax query missing ? separator

how you tested the change

Tested manually through the user interface.

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

fixes deleting EU conversion delete
@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 7, 2025

@adamkorynta adamkorynta merged commit 8fd000e into main Apr 8, 2025
9 checks passed
@MikeNeilson MikeNeilson deleted the bugfix/delete_unit_conversion branch June 1, 2025 20:47
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.

Cannot delete a row in the Engineering Unit Conversions page

3 participants