You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by mi...@apache.org on 2024/01/25 14:46:20 UTC

(superset) 03/08: fix: do not use lodash/memoize (#26709)

This is an automated email from the ASF dual-hosted git repository.

michaelsmolina pushed a commit to branch 3.0
in repository https://gitbox.apache.org/repos/asf/superset.git

commit 0d9a67261d9cc447681cef103a5777d7fa13b49f
Author: Evan Rusackas <ev...@preset.io>
AuthorDate: Mon Jan 22 12:36:43 2024 -0700

    fix: do not use lodash/memoize (#26709)
    
    (cherry picked from commit ef4878b845ea2d3de3c0ad83a4ebfc220d8f14f4)
---
 superset-frontend/src/utils/getControlsForVizType.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/superset-frontend/src/utils/getControlsForVizType.js b/superset-frontend/src/utils/getControlsForVizType.js
index ae48b8b0d8..8771d91dc7 100644
--- a/superset-frontend/src/utils/getControlsForVizType.js
+++ b/superset-frontend/src/utils/getControlsForVizType.js
@@ -17,12 +17,12 @@
  * under the License.
  */
 
-import memoize from 'lodash/memoize';
+import memoizeOne from 'memoize-one';
 import { isControlPanelSectionConfig } from '@superset-ui/chart-controls';
 import { getChartControlPanelRegistry } from '@superset-ui/core';
 import { controls } from '../explore/controls';
 
-const memoizedControls = memoize((vizType, controlPanel) => {
+const memoizedControls = memoizeOne((vizType, controlPanel) => {
   const controlsMap = {};
   (controlPanel?.controlPanelSections || [])
     .filter(isControlPanelSectionConfig)