[superset] branch master updated: fix: Clearing the currency format has no effect on the chart (#25238)

[michaelsmolina]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 6f4e63162f fix: Clearing the currency format has no effect on the 
chart (#25238)
6f4e63162f is described below

commit 6f4e63162faf91f263e02f3edfedaa95a35a6a91
Author: Michael S. Molina <70410625+michael-s-mol...@users.noreply.github.com>
AuthorDate: Mon Sep 11 08:26:43 2023 -0300

    fix: Clearing the currency format has no effect on the chart (#25238)
---
 .../src/explore/components/controls/CurrencyControl/CurrencyControl.tsx | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/superset-frontend/src/explore/components/controls/CurrencyControl/CurrencyControl.tsx
 
b/superset-frontend/src/explore/components/controls/CurrencyControl/CurrencyControl.tsx
index 5bbe271150..9c71e6edb5 100644
--- 
a/superset-frontend/src/explore/components/controls/CurrencyControl/CurrencyControl.tsx
+++ 
b/superset-frontend/src/explore/components/controls/CurrencyControl/CurrencyControl.tsx
@@ -107,6 +107,7 @@ export const CurrencyControl = ({
           onChange={(symbolPosition: string) => {
             onChange({ ...currency, symbolPosition });
           }}
+          onClear={() => onChange({ ...currency, symbolPosition: undefined })}
           value={currency?.symbolPosition}
           allowClear
           {...symbolSelectOverrideProps}
@@ -118,6 +119,7 @@ export const CurrencyControl = ({
           onChange={(symbol: string) => {
             onChange({ ...currency, symbol });
           }}
+          onClear={() => onChange({ ...currency, symbol: undefined })}
           value={currency?.symbol}
           allowClear
           allowNewOptions