Skip to content

Conversation

arikfr
Copy link
Member

@arikfr arikfr commented Dec 25, 2019

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

  • Refactor

Description

  • 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).

Related Tickets & Documents

#4099

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dashboards list includes dashboards with text widgets where you have no access to any other widgets

1 participant