Skip to content

Add all database access permission to 'Alpha' role#7271

Merged
mistercrunch merged 2 commits into
apache:masterfrom
mistercrunch:alpha_all_dbs
Apr 15, 2019
Merged

Add all database access permission to 'Alpha' role#7271
mistercrunch merged 2 commits into
apache:masterfrom
mistercrunch:alpha_all_dbs

Conversation

@mistercrunch
Copy link
Copy Markdown
Member

@mistercrunch mistercrunch commented Apr 10, 2019

Recently added a DatabaseFilter construct that allows to limit database list access per-role in SQL Lab. Without this fix, by default Alpha sees no database by default in their database dropdown list.

related to #7009

@request-info
Copy link
Copy Markdown

request-info Bot commented Apr 10, 2019

We would appreciate it if you could provide us with more info about this issue/pr!

@request-info request-info Bot added the requires:more-info Requires more information from author label Apr 10, 2019
@codecov-io
Copy link
Copy Markdown

codecov-io commented Apr 10, 2019

Codecov Report

Merging #7271 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #7271   +/-   ##
=======================================
  Coverage   64.72%   64.72%           
=======================================
  Files         425      425           
  Lines       20672    20672           
  Branches     2282     2282           
=======================================
  Hits        13380    13380           
  Misses       7167     7167           
  Partials      125      125
Impacted Files Coverage Δ
superset/security.py 75.65% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c38cb1...8d1fe67. Read the comment docs.

@mistercrunch mistercrunch merged commit 9dfa80d into apache:master Apr 15, 2019
@mistercrunch mistercrunch deleted the alpha_all_dbs branch April 15, 2019 16:22
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.34.0 First shipped in 0.34.0 labels Feb 28, 2024
qfcwell pushed a commit to qfcwell/superset that referenced this pull request May 12, 2026
* Add all database access permission to 'Alpha' role

* Grant all database access to Alpha
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels requires:more-info Requires more information from author 🚢 0.34.0 First shipped in 0.34.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants