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

Datasources: add new tab and feature toggle #25663

Merged

Conversation

AgnesToulet
Copy link
Contributor

What this PR does / why we need it:
Add a new tab and a feature toggle to enable a new Enterprise feature (Datasource Insights)

@AgnesToulet AgnesToulet added this to the 7.1 milestone Jun 17, 2020
@AgnesToulet AgnesToulet requested review from sakjur, alexanderzobnin and a team June 17, 2020 12:56
@AgnesToulet AgnesToulet self-assigned this Jun 17, 2020
@AgnesToulet AgnesToulet requested review from torkelo and tskarhed and removed request for a team June 17, 2020 12:56
Copy link
Contributor

@alexanderzobnin alexanderzobnin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it OK that we have 404 on the insights page? Looks like there's no route in grafana to that pages. Or it will be implemented in future PR?

@AgnesToulet
Copy link
Contributor Author

Yes, I just created the other PR in the Enterprise repo.

@alexanderzobnin
Copy link
Contributor

Maybe it makes sense to show this only in Enterprise version to prevent confused 404 if feature toggled in open source?

@AgnesToulet
Copy link
Contributor Author

You shouldn't be able to see the Insights tab without an Enterprise license even if the feature is enabled.

@alexanderzobnin alexanderzobnin self-requested a review June 17, 2020 15:09
@AgnesToulet AgnesToulet merged commit 4853770 into grafana:master Jun 18, 2020
aknuds1 pushed a commit that referenced this pull request Jun 30, 2020
* Datasources: add insights tab

* Config: add feature flag
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants