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

add to dashboard bugs #10759

Closed
1 of 3 tasks
camerondeleone opened this issue Jul 13, 2022 · 2 comments
Closed
1 of 3 tasks

add to dashboard bugs #10759

camerondeleone opened this issue Jul 13, 2022 · 2 comments
Assignees
Labels
bug Something isn't working right feature/dashboards Feature Tag: Dashboards feature/permissions Feature Tag: Permissions

Comments

@camerondeleone
Copy link
Contributor

camerondeleone commented Jul 13, 2022

Bug description

There are a couple related bugs here.

  1. This path results in unexpected results: Add insight to dashboard, click into dashboard to edit (switch from funnel to trend), save insight --> results not updated on dashboard even though they are reflected when viewing individual insight)
  2. Clicking into recently edited insight above and adding to dash, I accidentally added it to a dashboard I didn't want to (first on the list, "The Survey"), but then could not remove it. Looks like I may not have permission to do this, but was able to add something via the 'add to dashboard' route from the individual insight regardless.

How to reproduce

First issue:

  1. Create an insight and add to dashboard
  2. click through from dash to 'edit' and change the insight type, then save
  3. view that in dash - it's not updated

Second issue:

  1. on an insight, click add to dashboard and select one you don't have edit permission on
  2. you should be able to add the insight from this route, but then cannot remove it
  3. Here is a culprit insight and dash (and please remove if you have permission)

Here is a quick video https://posthog.slack.com/archives/C01RJ7T212S/p1657708603632139

Environment

  • PostHog Cloud
  • self-hosted PostHog (ClickHouse-based), version/commit: please provide
  • self-hosted PostHog (Postgres-based, legacy), version/commit: please provide

Additional context

Thank you for your bug report – we love squashing them!

@camerondeleone camerondeleone added bug Something isn't working right feature/dashboards Feature Tag: Dashboards feature/permissions Feature Tag: Permissions labels Jul 13, 2022
@pauldambra pauldambra self-assigned this Jul 13, 2022
@pauldambra
Copy link
Member

One part of this is in #10791. We weren't checking that someone had permission on a dashboard before adding an insight to it. But operations after that then took account of the new permissions. So we can fix that...

But you're a staff/admin user so the next question is whether you should still have had permissions after adding the insight...

@pauldambra
Copy link
Member

2 is fixed. 1 is a duplicate of #10263

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working right feature/dashboards Feature Tag: Dashboards feature/permissions Feature Tag: Permissions
Projects
Development

No branches or pull requests

2 participants