-
Notifications
You must be signed in to change notification settings - Fork 654
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
Redirect to list on delete table vis #2697
Conversation
Depends on user feature flag of new dashboard
@xavijam code review please. don't think it's worth adding a test because it's a minimal change and non-critical code path, and since setting up a test case would require setting up a lot of stuff. :/ |
Since new dashboard will render it as datasets as fallback (already done)
if (self.user.featureEnabled('new_dashboard')) { | ||
window.location = self.user.viewUrl().dashboard().datasets(); | ||
} else { | ||
window.location = cdb.config.prefixUrl() + "/dashboard/tables" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd only redirect to dashboard/tables because new-dashboard knows how to display datasets although url says tables
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 069ab45
👍 |
Frontend tests were OK 👍 (details) |
…-table-vis Redirect to list on delete table vis
Fixes #2568
Depending on if user has new dashboard or not he/she is redirected to /dashboard/datasets or /dashboard/tables