-
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
Plugins: Add dependency column in version table #73991
Plugins: Add dependency column in version table #73991
Conversation
@@ -26,6 +26,7 @@ export const VersionList = ({ versions = [], installedVersion }: Props) => { | |||
<tr> | |||
<th>Version</th> | |||
<th>Last updated</th> | |||
<th>Dependency</th> |
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.
<th>Dependency</th> | |
<th>Grafana Dependency</th> |
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 think this makes it more understandable by users 😄
- Consider cases where there are additional dependencies like with synthetic monitoring, we're only showing the Grafana version dependency so we should make that clear.
![Screenshot 2023-08-29 at 14 37 10](https://private-user-images.githubusercontent.com/19860021/264037661-45cac2bf-918b-4edd-a3a8-612c2bf73992.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjAyNTYxMjIsIm5iZiI6MTcyMDI1NTgyMiwicGF0aCI6Ii8xOTg2MDAyMS8yNjQwMzc2NjEtNDVjYWMyYmYtOTE4Yi00ZWRkLWEzYTgtNjEyYzJiZjczOTkyLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA2VDA4NTAyMlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTU4MGNmMzQ0M2UwODg4YzFhZjhlOWY5NmMzNWNkM2QyMTMzNjc3NmYwZjQyYzc0NjBjZmY5NDZlMTlkOTUyMDImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.UwnXXWSbO_KHSezMIxSYS68lU272PmA5aHXtsSYaEW4)
I considered Grafana Version but that might cause confusion with the deprecated grafanaVersion
in plugin.json
and Grafana Version Dependency which felt too wordy.
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.
Nice work on this and fast turnaround 🚀 Please see comments for suggested change
I would like to incorporate this into our documentation on plugin management... but we would need to add a lot more before this specific addition would make sense, so don't worry for now, I will raise a follow up issue.
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.
Tested locally and LGTM from user perspective 🥇
* Plugins: add dependency column in version table * Closes grafana#44052
* Plugins: add dependency column in version table * Closes #44052
What is this feature?
Introduce Dependency column in Version History table
Why do we need this feature?
This is necessary to give better info about a plugin previous versions
Who is this feature for?
Plugins users
Which issue(s) does this PR fix?:
#44052
Special notes for your reviewer:
Please check that: