You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by pk...@apache.org on 2022/04/08 01:45:55 UTC
[superset] branch master updated: feat: add success toast to alerts and reports (#19482)
This is an automated email from the ASF dual-hosted git repository.
pkdotson 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 44e3103fc9 feat: add success toast to alerts and reports (#19482)
44e3103fc9 is described below
commit 44e3103fc9029edd4c7fd1c8402607eecebeb1b3
Author: Phillip Kelley-Dotson <pk...@yahoo.com>
AuthorDate: Thu Apr 7 18:45:35 2022 -0700
feat: add success toast to alerts and reports (#19482)
* feat: add success toast to alerts and reports
* remove console
* add suggestions
---
superset-frontend/src/views/CRUD/alert/AlertReportModal.tsx | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/superset-frontend/src/views/CRUD/alert/AlertReportModal.tsx b/superset-frontend/src/views/CRUD/alert/AlertReportModal.tsx
index 6323edf3a0..7a68b2663f 100644
--- a/superset-frontend/src/views/CRUD/alert/AlertReportModal.tsx
+++ b/superset-frontend/src/views/CRUD/alert/AlertReportModal.tsx
@@ -73,6 +73,7 @@ type SelectValue = {
};
interface AlertReportModalProps {
+ addSuccessToast: (msg: string) => void;
addDangerToast: (msg: string) => void;
alert?: AlertObject | null;
isReport?: boolean;
@@ -402,6 +403,7 @@ const AlertReportModal: FunctionComponent<AlertReportModalProps> = ({
show,
alert = null,
isReport = false,
+ addSuccessToast,
}) => {
const conf = useCommonConf();
const allowedNotificationMethods: NotificationMethodOption[] =
@@ -555,6 +557,8 @@ const AlertReportModal: FunctionComponent<AlertReportModalProps> = ({
return;
}
+ addSuccessToast(t(`${data.type} updated`));
+
if (onAdd) {
onAdd();
}
@@ -569,6 +573,8 @@ const AlertReportModal: FunctionComponent<AlertReportModalProps> = ({
return;
}
+ addSuccessToast(t(`${data.type} updated`));
+
if (onAdd) {
onAdd(response);
}