You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by vi...@apache.org on 2021/08/10 05:01:34 UTC
[superset] 24/25: fix: change Alert Permissions (#16118)
This is an automated email from the ASF dual-hosted git repository.
villebro pushed a commit to branch 1.3
in repository https://gitbox.apache.org/repos/asf/superset.git
commit 80d09e63b21a6d5b650e435779f017c33e3fc0b5
Author: AAfghahi <48...@users.noreply.github.com>
AuthorDate: Mon Aug 9 16:33:04 2021 -0400
fix: change Alert Permissions (#16118)
* added google alert
* reworked permissions
(cherry picked from commit 606a7bf429edf28c410ee57aff1b5cbb062bd88f)
---
superset-frontend/src/dashboard/components/Header/index.jsx | 2 +-
superset-frontend/src/explore/components/ExploreChartHeader.jsx | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/superset-frontend/src/dashboard/components/Header/index.jsx b/superset-frontend/src/dashboard/components/Header/index.jsx
index 8d396e3..7069954 100644
--- a/superset-frontend/src/dashboard/components/Header/index.jsx
+++ b/superset-frontend/src/dashboard/components/Header/index.jsx
@@ -417,7 +417,7 @@ class Header extends React.PureComponent {
const roles = Object.keys(user.roles || []);
const permissions = roles.map(key =>
user.roles[key].filter(
- perms => perms[0] === 'can_add' && perms[1] === 'AlertModelView',
+ perms => perms[0] === 'menu_access' && perms[1] === 'Manage',
),
);
return permissions[0].length > 0;
diff --git a/superset-frontend/src/explore/components/ExploreChartHeader.jsx b/superset-frontend/src/explore/components/ExploreChartHeader.jsx
index 0613f61..7b19d22 100644
--- a/superset-frontend/src/explore/components/ExploreChartHeader.jsx
+++ b/superset-frontend/src/explore/components/ExploreChartHeader.jsx
@@ -200,7 +200,7 @@ export class ExploreChartHeader extends React.PureComponent {
const roles = Object.keys(user.roles || []);
const permissions = roles.map(key =>
user.roles[key].filter(
- perms => perms[0] === 'can_add' && perms[1] === 'AlertModelView',
+ perms => perms[0] === 'menu_access' && perms[1] === 'Manage',
),
);
return permissions[0].length > 0;