Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[explore] CSV Export permission is not consistent between Dashboard and Explore window #13711

Closed
3 tasks done
duynguyenhoang opened this issue Mar 19, 2021 · 0 comments · Fixed by #13713
Closed
3 tasks done
Labels
#bug Bug report

Comments

@duynguyenhoang
Copy link
Contributor

Expected results

CSV Export permission must be consistent. When user doesn't have permission "Can CSV on Superset", he/she won't be able to Export CSV.
There is no Export CSV menu on Dashboard and on Explore page in this case.

Actual results

"Can CSV on Superset" is not assigned to user.
From Dashboard, user is unable to Export CSV but in Chart user can.

Screenshots

There is no Export CSV menu in Dashboard screen
Screenshot_20210320_002214

There is CSV button and we can Export CSV
Screenshot_20210320_002357

How to reproduce the bug

  1. A user with normal role, can access Chart/Dashboard and Explore page
  2. But he/she doesn't have "Can CSV on Superset" permission
  3. Go to any Dashboard, there is no "Export CSV" menu in any chart Context menu
  4. View chart in Explore
  5. On the right top pannel, there is "Export to CSV format" => Which is wrong.

Environment

  • superset version: Master, SHA: 2b0b3f2f
  • python version: Python 3.7.9
  • node.js version: v14.16.0

Checklist

Make sure to follow these steps before submitting your issue - thank you!

  • I have checked the superset logs for python stacktraces and included it here as text if there are any.
  • I have reproduced the issue with at least the latest released version of superset.
  • I have checked the issue tracker for the same issue and I haven't found one similar.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#bug Bug report
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant