Add metricsEnableSavingPinsToggled action and related reducer

Author: roseayeon

tensorboard/webapp/metrics/actions/index.ts

@@ -281,5 +281,9 @@ export const metricsClearAllPinnedCards = createAction(
   '[Metrics] Clear all pinned cards'
 );
 
+export const metricsEnableSavingPinsToggled = createAction(
+  '[Metrics] Enable Saving Pins Toggled'
+);
+
 // TODO(jieweiwu): Delete after internal code is updated.
 export const stepSelectorTimeSelectionChanged = timeSelectionChanged;

tensorboard/webapp/metrics/store/metrics_reducers.ts

@@ -933,6 +933,19 @@ const reducer = createReducer(
       },
     };
   }),
+  on(actions.metricsEnableSavingPinsToggled, (state) => {
+    const nextSavingPinsEnabled = !(
+      state.settingOverrides.savingPinsEnabled ??
+      state.settings.savingPinsEnabled
+    );
+    return {
+      ...state,
+      settingOverrides: {
+        ...state.settingOverrides,
+        savingPinsEnabled: nextSavingPinsEnabled,
+      },
+    };
+  }),
   on(
     actions.multipleTimeSeriesRequested,
     (

tensorboard/webapp/metrics/store/metrics_reducers_test.ts

@@ -1232,6 +1232,21 @@ describe('metrics reducers', () => {
       expect(thirdState.settings.hideEmptyCards).toBe(false);
       expect(thirdState.settingOverrides.hideEmptyCards).toBe(false);
     });
+
+    it('changes savingPinsEnabled on metricsEnableSavingPinsToggled', () => {
+      const prevState = buildMetricsState({
+        settings: buildMetricsSettingsState({
+          savingPinsEnabled: true,
+        }),
+        settingOverrides: {},
+      });
+      const nextState = reducers(
+        prevState,
+        actions.metricsEnableSavingPinsToggled()
+      );
+      expect(nextState.settings.savingPinsEnabled).toBe(true);
+      expect(nextState.settingOverrides.savingPinsEnabled).toBe(false);
+    });
   });
 
   describe('loading time series data', () => {