Skip to content

fix: update dependencies to get rid of d2 init warnings#1415

Merged
KaiVandivier merged 2 commits intomasterfrom
fix/d2-init-warnings
Jul 8, 2025
Merged

fix: update dependencies to get rid of d2 init warnings#1415
KaiVandivier merged 2 commits intomasterfrom
fix/d2-init-warnings

Conversation

@kabaros
Copy link
Copy Markdown
Collaborator

@kabaros kabaros commented Jun 24, 2025

changes prop-types to a dependency rather than a peer dependency, similar to what we do in UI.

Implements https://dhis2.atlassian.net/browse/LIBS-771

related to dhis2/app-platform#927

Comment thread runtime/package.json Outdated
Comment on lines +42 to +43
"react": "^16 || ^18",
"react-dom": "^16 || ^18"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This PR mostly looks good!

My one comment is that I think it makes sense to keep the 16.8.6 floor for React -- 16.8.0 introduces hooks, so this library won't work with lower versions of v16

It's unlikely to be encountered, but technically correct I think 😅

the version where react hooks were introduced
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jul 8, 2025

@KaiVandivier KaiVandivier merged commit ec044f3 into master Jul 8, 2025
16 checks passed
@KaiVandivier KaiVandivier deleted the fix/d2-init-warnings branch July 8, 2025 14:55
dhis2-bot added a commit that referenced this pull request Jul 8, 2025
## [3.14.5](v3.14.4...v3.14.5) (2025-07-08)

### Bug Fixes

* update dependencies to get rid of d2 init warnings ([#1415](#1415)) ([ec044f3](ec044f3))
@dhis2-bot
Copy link
Copy Markdown
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants