-
Notifications
You must be signed in to change notification settings - Fork 11.7k
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
Explore: Remove exploreMixedDatasource feature toggle #71534
Conversation
(Open the links below in a new tab to go to the correct steps)
|
Backend code coverage report for PR #71534 |
Frontend code coverage report for PR #71534
|
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.
Looks great to me 👍
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.
public/app/core/utils/explore.ts
Outdated
@@ -75,21 +75,6 @@ export async function getExploreUrl(args: GetExploreUrlArguments): Promise<strin | |||
.filter((t) => t.datasource?.uid !== ExpressionDatasourceUID); | |||
let url: string | undefined; | |||
// if the mixed datasource is not enabled for explore, choose only one datasource |
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.
// if the mixed datasource is not enabled for explore, choose only one datasource |
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.
There's a couple mentions of allowMixed
in this that I can't make suggested changes on bc they're too far from the other changes.
What is this feature?
Removes the
exploreMixedDatasource
feature flag. The flag was enabled by default in 10.0.0 and no issues has been reported about itWhich issue(s) does this PR fix?:
Fixes #52417
Special notes for your reviewer:
Please check that:
a couple of now irrelevant tests have been removed. Please test that: