-
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
Upgrade plugin dependencies to React 18 #212
Upgrade plugin dependencies to React 18 #212
Conversation
79cabe2
to
f30ffb5
Compare
6cf2694
to
ec39537
Compare
* try using github_token * updating commands to match readme and also to kick the drone build
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.
did a very brief test and the datasource seems to load ok for me! Let me know if there's anything in particular that would be good to test!
@@ -26,18 +26,18 @@ | |||
"@babel/core": "^7.16.7", | |||
"@emotion/css": "^11.1.3", | |||
"@grafana/aws-sdk": "0.1.2", | |||
"@grafana/data": "8.4.11", | |||
"@grafana/e2e": "8.4.11", | |||
"@grafana/data": "10.0.3", |
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.
Let's make sure we test this change with 9.4 for AMG, maybe we can reach out to folks at AWS to test on their end
selectedEntityValue && selectedComponentValue | ||
? { | ||
[DataBindingLabelKeys.entityId]: selectedEntityValue, |
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.
Just curious why were these changes necessary?
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.
This change is required due to the change in the definition for selectedDataBinding in iot-app-kit.
package.json
Outdated
@@ -118,6 +119,12 @@ | |||
"debug": "^4.0.0", | |||
"underscore": "^1.12.1", | |||
"d3-color": "^3.1.0", | |||
"log4js": "^6.4.0" | |||
"log4js": "^6.4.0", | |||
"react-dnd": "^16.0.1", |
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 like react-dnd@16.0.1
is the version that iot-app-kit/*
v7 uses so i don't think we need the resolution here.
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.
Good catch!
CHANGELOG.md
Outdated
@@ -1,8 +1,13 @@ | |||
# Changelog | |||
|
|||
## 1.8.1 |
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.
## 1.8.1 | |
## 1.8.0 |
package.json
Outdated
@@ -1,9 +1,9 @@ | |||
{ | |||
"name": "grafana-iot-twinmaker-app", | |||
"version": "1.7.1", | |||
"version": "1.8.1", |
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.
"version": "1.8.1", | |
"version": "1.8.0", |
…/github.com/grafana/grafana-iot-twinmaker-app into 205-upgrade-plugin-dependencies-to-react-18
What this PR does / why we need it:
This upgrades plugin dependencies to React 18.
Which issue(s) this PR fixes:
Fixes #205
Special notes for your reviewer:
Context: https://raintank-corp.slack.com/archives/C01LVB0CYTW/p1692336771498409