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

Feat: Changing link destination for get more plugins #63517

Merged
merged 3 commits into from Mar 7, 2023

Conversation

tolzhabayev
Copy link
Contributor

@tolzhabayev tolzhabayev commented Feb 21, 2023

What is this feature?

Changing the link destination for "find more plugins" from external to internal plugin catalog or connections depending on if the feature flag is active or not.

If feature flag dataConnectionsConsole is active it links to /connections/connect-data?cat=data-source if not it links to /plugins?filterBy=all&filterByType=datasource

Why do we need this feature?

Keep users in grafana catalog, less confusion when searching for new plugins

SCR-20230221-o33

@tolzhabayev tolzhabayev marked this pull request as ready for review February 21, 2023 16:23
@tolzhabayev tolzhabayev requested a review from a team as a code owner February 21, 2023 16:23
@tolzhabayev tolzhabayev requested review from joshhunt and yaelleC and removed request for a team February 21, 2023 16:23
@tolzhabayev tolzhabayev added this to the 9.5.0 milestone Feb 21, 2023
@tolzhabayev tolzhabayev added no-backport Skip backport of PR no-changelog Skip including change in changelog/release notes add to changelog and removed no-changelog Skip including change in changelog/release notes labels Feb 21, 2023
@tolzhabayev tolzhabayev changed the title Changing link destination for get more plugins Feat: Changing link destination for get more plugins Feb 21, 2023
@@ -351,7 +351,7 @@ lerna.json @grafana/frontend-ops
/public/app/features/connections/ @grafana/plugins-platform-frontend
/public/app/features/correlations/ @grafana/explore-squad
/public/app/features/dashboard/ @grafana/dashboards-squad
/public/app/features/datasources/ @grafana/user-essentials
/public/app/features/datasources/ @grafana/plugins-platform-frontend
Copy link
Contributor Author

Choose a reason for hiding this comment

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

switching ownership to plugins platform

@tolzhabayev tolzhabayev self-assigned this Feb 21, 2023
@tolzhabayev
Copy link
Contributor Author

fyi @mjseaman

Copy link
Contributor

@leventebalogh leventebalogh left a comment

Choose a reason for hiding this comment

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

LGTM, left one comment. 👍

Copy link
Contributor

@leventebalogh leventebalogh left a comment

Choose a reason for hiding this comment

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

🚢🇮🇹

@tolzhabayev tolzhabayev merged commit 20b79b4 into main Mar 7, 2023
@tolzhabayev tolzhabayev deleted the feat/change-get-more-plugins-link branch March 7, 2023 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants