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

kie-issues#1779: Update deprecated react-json-view dependency #2870

Merged
merged 5 commits into from
Jan 24, 2025

Conversation

kbowers-ibm
Copy link
Contributor

@kbowers-ibm kbowers-ibm commented Jan 23, 2025

Closes apache/incubator-kie-issues#1779
Replacing the deprecated https://github.com/mac-s-g/react-json-view with https://github.com/microlinkhq/react-json-view (recommended by author of the deprecated dependency)

@jomarko jomarko self-requested a review January 23, 2025 15:22
@@ -37,6 +37,7 @@
"@kie-tools/i18n-common-dictionary": "workspace:*",
"@kie-tools/runtime-tools-components": "workspace:*",
"@kie-tools/runtime-tools-shared-gateway-api": "workspace:*",
"@microlink/react-json-view": "^1.24.0",
Copy link
Member

Choose a reason for hiding this comment

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

It looks like this dependency is not used in this package; maybe it could be removed?

Copy link
Contributor

@ljmotta ljmotta left a comment

Choose a reason for hiding this comment

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

@kbowers-ibm Thanks for this change! @microlink/react-json-view is a fork of the previous library. I don't think we should have any problems with this change.

@tiagobento tiagobento added the area:dependencies Pull requests that update a dependency file label Jan 23, 2025
@tiagobento
Copy link
Contributor

@kbowers-ibm Conflicts... 🙈

@kbowers-ibm kbowers-ibm merged commit bbf18e9 into apache:main Jan 24, 2025
15 checks passed
@kbowers-ibm kbowers-ibm deleted the kie-issues#1779 branch January 24, 2025 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update deprecated react-json-view dependency
5 participants