Refine permissions usage in Redash to allow for guest users #4492

arikfr · 2019-12-25T15:59:19Z

What type of PR is this? (check all applicable)

Make sure we don't list dashboards you only have access to the text widgets in them (fixes Dashboards list includes dashboards with text widgets where you have no access to any other widgets #4099). It does mean that if someone creates a dashboard with only a text widget in it, only they will see it listed. Doesn't feel like a big issue.
AppHeader: some elements were rendered when they aren't supposed to (like Create button or Query Snippets link).
Allow using Query Results APIs with execute_query permission (along with view_query).

…ions.

Closes #4099.

arikfr added 3 commits December 25, 2019 17:57

Allow executing query with either view_query or execute_query permiss…

955c047

…ions.

Render AuthHeader according to permissions.

968a392

Don't return dashboards where you only have access to textbox widget.

46dc775

Closes #4099.

arikfr merged commit 30bc1e2 into master Dec 30, 2019

arikfr deleted the guest-mode branch December 30, 2019 08:07

weekly-digest bot mentioned this pull request Dec 30, 2019

Weekly Digest (23 December, 2019 - 30 December, 2019) #4508

Closed

weekly-digest bot mentioned this pull request Jan 6, 2020

Weekly Digest (30 December, 2019 - 6 January, 2020) #4523

Closed

arikfr mentioned this pull request Feb 23, 2020

Dashboard visibility problem #4671

Closed