dc079f0d8d
## Description When we load API editor, we notice an update call without any changes. This happens because of the Headers and Params of the API editor adding empty key value fields. These fields are removed by the RestAPITransformer but the save keeps happening and we update the action state without the empty fields. When we load the API again, the API editor notices that no fields exists and adds them back again. This thus lead to excessive saves of the API to the server without any benefit. We will now check for this case before hand and avoid starting a save when form adds these empty key value fields ## Automation /ok-to-test tags="@tag.Datasource" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/14075955172> > Commit: b11f85f3ee716b2042331a55b670e292516755d6 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=14075955172&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.Datasource` > Spec: > <hr>Wed, 26 Mar 2025 05:59:23 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Bug Fixes** - Refined API action updates by filtering out empty entries during save operations, ensuring only valid data is processed and improving overall data integrity. - **New Features** - Introduced a utility function to check for empty key-value pairs, enhancing the logic for action property updates. <!-- end of auto-generated comment: release notes by coderabbit.ai --> |
||
|---|---|---|
| .. | ||
| client | ||
| server | ||
| util | ||