You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the chart wizard, the "view in SQL Lab" button only appears in the context menu for the dataset if the user specifically has the sql_lab role, rather than depending on the relevant permission associated with this action: [SQL Lab].[menu_access]. Update this check to look for the permission instead of the role. Since the sql_lab role has that permission, it will continue to work as normal, but with a more appropriate check.
This fixes for the case where this permission has been assigned to a different role.
How to reproduce the bug
Add the [SQL Lab].[menu_access] role to the Gamma role, or some other role associated with a user
As a Gamma user, view a dataset in the chart explorer
Click on the three dots next to the dataset name in the top left
See that View in SQL Lab button is missing
Expected results
User should see the option to go to SQL Lab as they have SQL Lab access.
Actual results
Button is missing on the frontend.
Environment
(please complete the following information):
browser type and version: N/A
superset version: 2.1.0
python version: N/A
node.js version: N/A
any feature flags active:
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.
Additional context
I discussed this with @villebro who made the original change to check the role and agreed on the solution being to check the permission I named here instead; I'll raise a PR for this issue now as I already have this fixed and tested in our fork.
The text was updated successfully, but these errors were encountered:
In the chart wizard, the "view in SQL Lab" button only appears in the context menu for the dataset if the user specifically has the
sql_lab
role, rather than depending on the relevant permission associated with this action:[SQL Lab].[menu_access]
. Update this check to look for the permission instead of the role. Since thesql_lab
role has that permission, it will continue to work as normal, but with a more appropriate check.This fixes for the case where this permission has been assigned to a different role.
How to reproduce the bug
[SQL Lab].[menu_access]
role to theGamma
role, or some other role associated with a userView in SQL Lab
button is missingExpected results
User should see the option to go to SQL Lab as they have SQL Lab access.
Actual results
Button is missing on the frontend.
Environment
(please complete the following information):
N/A
2.1.0
N/A
N/A
Checklist
Make sure to follow these steps before submitting your issue - thank you!
Additional context
I discussed this with @villebro who made the original change to check the role and agreed on the solution being to check the permission I named here instead; I'll raise a PR for this issue now as I already have this fixed and tested in our fork.
The text was updated successfully, but these errors were encountered: