From f47332a129b413086a84d641eab7e46334759bb2 Mon Sep 17 00:00:00 2001 From: Ankita Kinger Date: Tue, 24 Jun 2025 13:54:49 +0530 Subject: [PATCH] fix: Removing a part of code as its not updating the API params dependency correctly in Evaluations (#41019) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Description Removing a part of code as its not updating the API params dependency correctly in Evaluations Fixes [#41021](https://github.com/appsmithorg/appsmith/issues/41021) ## Automation /ok-to-test tags="@tag.All" ### :mag: Cypress test results > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: > Commit: 0c78c53a95dbdf37c0e157e0cb15fdce9aa6674c > Cypress dashboard. > Tags: `@tag.All` > Spec: >
Mon, 23 Jun 2025 16:31:24 UTC ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No ## Summary by CodeRabbit - **Refactor** - Updated the behavior of the API configuration form to streamline how path and query parameters are handled during autofill. The form now updates the visible path field without triggering additional background updates. --- app/client/src/sagas/ApiPaneSagas.ts | 9 --------- 1 file changed, 9 deletions(-) diff --git a/app/client/src/sagas/ApiPaneSagas.ts b/app/client/src/sagas/ApiPaneSagas.ts index 29a42fbfd3..0f9aa00f1c 100644 --- a/app/client/src/sagas/ApiPaneSagas.ts +++ b/app/client/src/sagas/ApiPaneSagas.ts @@ -124,15 +124,6 @@ function* syncApiParamsSaga( `${currentPath}${paramsString}`, ), ); - - // Also update the action property to ensure the path is updated in the action - yield put( - setActionProperty({ - actionId: actionId, - propertyName: "actionConfiguration.path", - value: `${currentPath}${paramsString}`, - }), - ); } }