Move to navigate route by default (#1829)

Co-authored-by: Evgenij Shangin <[email protected]>
datalens-tech · Nov 22, 2024 · 605cb2f · 605cb2f
1 parent c728cf7
commit 605cb2f
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 5 deletions.
diff --git a/src/server/components/resolve-entry-by-link.ts b/src/server/components/resolve-entry-by-link.ts
@@ -30,6 +30,7 @@ async function resolveEntryByLink({
         const idOrKeyOrReport =
             url.pathname
                 .replace(/^\/navigation\//, '')
+                .replace(/^\/navigate\//, '')
                 .replace(/^\/wizard\/preview\//, '')
                 .replace(
                     /^(\/(ChartPreview|preview))?(\/(ChartWizard|wizard|ChartEditor|editor))?\/?/,

diff --git a/src/ui/libs/DatalensChartkit/menu/MenuItems.tsx b/src/ui/libs/DatalensChartkit/menu/MenuItems.tsx
@@ -176,7 +176,11 @@ export const getEditMenuItem = ({
             window.open(
                 (dataProvider || chartsDataProvider)?.getGoAwayLink(
                     {loadedData, propsData},
-                    {idPrefix: '/navigation/'},
+                    {
+                        idPrefix: Utils.isEnabledFeature(Feature.UseNavigation)
+                            ? '/navigation/'
+                            : '/navigate/',
+                    },
                 ),
             );
         }),

diff --git a/src/ui/libs/DatalensChartkit/modules/menu/menu.ts b/src/ui/libs/DatalensChartkit/modules/menu/menu.ts
@@ -58,9 +58,6 @@ export const getChartkitMenuByType = (props?: GetChartkitMenuByType) => {
             },
             [MenuItemsIds.EDIT]: {
                 title: i18n('dash.chartkit-menu.view', 'button_edit'),
-                action: ({propsData: {id, params}}) => {
-                    window.open(`/navigate/${id}?${stringify(params)}`);
-                },
                 ...isEditVisible,
             },
             [MenuItemsIds.EXPORT]: {

diff --git a/src/ui/units/dash/modules/helpers.ts b/src/ui/units/dash/modules/helpers.ts
@@ -177,7 +177,7 @@ export function addOperationForValue(args: {
 export function getChartEditLink(entryId: string) {
     return Utils.isEnabledFeature(Feature.UseNavigation)
         ? `${window.DL.endpoints.charts}/navigation/${entryId}`
-        : `${window.DL.endpoints.wizard}/${entryId}`;
+        : `/navigate/${entryId}`;
 }
 
 export function deepAssign(...args: any) {