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.

Sign up for GitHub

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

Jump to bottom

Warn the user when deleting visualization used in dashboard #120

Open
arikfr opened this issue Mar 4, 2014 · 0 comments
Open

Warn the user when deleting visualization used in dashboard #120

arikfr opened this issue Mar 4, 2014 · 0 comments
Labels
Frontend

Comments

@arikfr
Copy link
Member

arikfr commented Mar 4, 2014

Before deleting a visualization we need to make sure that it isn't used in dashboards (meaning, no widgets reference it).

  1. If it's used in widgets, show the user a warning: "This visualization is used in X dashboards. Are you sure you want to delete it? (deleting the visualization will remove it from the dashboard)". If the user confirms, we need to delete the widgets too.
  2. If it's not used in widgets, then business as usual (just delete).
@arikfr arikfr added the Frontend label Mar 4, 2014
@arikfr arikfr added this to the v0.3 milestone Mar 4, 2014
arikfr added a commit that referenced this issue Apr 29, 2014
@arikfr
Workaround for cases when widget is missing but referenced in a dashb…
a71b99a 
…oard layout (re. #120)
arikfr added a commit that referenced this issue Apr 29, 2014
@arikfr
Merge pull request #194 from EverythingMe/feature_markdown_widget
855aecd 
Several small fixes (#186, #120, #174)
christophervalles pushed a commit to hailocab/redash that referenced this issue May 8, 2014
@arikfr
Workaround for cases when widget is missing but referenced in a dashb…
9f2fc1f 
…oard layout (re. getredash#120)
christophervalles pushed a commit to hailocab/redash that referenced this issue May 8, 2014
@arikfr
Merge pull request getredash#194 from EverythingMe/feature_markdown_w…
cdf6a19 
…idget

Several small fixes (getredash#186, getredash#120, getredash#174)
@arikfr arikfr modified the milestones: Next, v0.4 Jul 20, 2014
@arikfr arikfr removed this from the Next milestone Dec 10, 2015
washort pushed a commit to washort/redash that referenced this issue Jul 25, 2017
@alison985
Merge pull request getredash#120 from alison985/115_fix_athena_direct…
d7b6409 
…_query_failure

115 fix athena direct query failure
dairyo pushed a commit to KiiCorp/redash that referenced this issue Mar 1, 2019
@arikfr
Workaround for cases when widget is missing but referenced in a dashb…
5e65bed 
…oard layout (re. getredash#120)
dairyo pushed a commit to KiiCorp/redash that referenced this issue Mar 1, 2019
@arikfr
Workaround for cases when widget is missing but referenced in a dashb…
5a79e67 
…oard layout (re. getredash#120)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Frontend
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@arikfr @amirnissim