-
Notifications
You must be signed in to change notification settings - Fork 13
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
ConfigEditor
updates
#72
Conversation
- Replace legacy components - Update tests - Add version utils - Add grafana/experimental
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
435bf0e
to
145c968
Compare
This comment was marked as outdated.
This comment was marked as outdated.
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 good! I'd be keen to see if the semver
package can be used instead of having our own code for this, it's so commonly used that it's probably already in our deps tree somewhere so we're probably OK just to depend on it ourselves.
This comment was marked as outdated.
This comment was marked as outdated.
This reverts commit 2f90bf3.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Use the following command to run this PR with Docker at http://localhost:3000:
|
569a391
to
1138e71
Compare
Use the following command to run this PR with Docker at http://localhost:3000:
|
Use the following command to run this PR with Docker at http://localhost:3000:
|
Use the following command to run this PR with Docker at http://localhost:3000:
|
For standardisation across Grafana we're updating datasource config editors to be more consistent. This is a part of https://github.com/grafana/oss-plugin-partnerships/issues/225.
This is a breaking change as the plugin now has a minimum Grafana dependency of 9.5.0
The config editor has been updated to replace
InlineField
withField
and the config editor has been refactored to use the new config editor structural components. There is also now individual validation on required fields. Node version has also been upgraded to 18.Fixes grafana/oss-plugin-partnerships#254