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

[dependency] UI dependency update #1906

Open
wants to merge 67 commits into
base: dev
Choose a base branch
from
Open

Conversation

veronikaslc
Copy link
Contributor

@veronikaslc veronikaslc commented Feb 26, 2025

Updated dependencies:

React from v. 18 to v. 19
react-router from v. 5 to v. 7
node from v. 16 to v. 22
yarn
webpack
babel
@mui/x-date-pickers from v. 6 to v. 7
@mui/lab from v. 5 to v. 6
@mui/icons-material from v. 5 to v. 6
@mui/material from v. 5 to v. 6
@mui/styles from v. 5 to v. 6
@uiw/react-md-editor from v. 3 to v. 4
css-loader from v. 6 to v. 7
eslint from v. 8 to v. 9
history from v. 4 to v. 5
react-to-print from v. 2 to v. 3
style-loader from v. 3 to v. 4
uuid from v. 9 to v. 11
yup from v. 0 to v. 1
eslint-plugin-react
recharts
cornerstone-wado-image-loader
react-number-format
react-google-autocomplete
google-palette

Removed unused dependencies: semantic-ui-css, semantic-ui-react, set-blocking, formik-material-fields

@veronikaslc veronikaslc added dependencies Pull requests that update a dependency file large PR labels Feb 26, 2025
@marta- marta- requested review from sdumitriu and mdpham March 3, 2025 16:27
@sdumitriu

This comment was marked as resolved.

@sdumitriu

This comment was marked as resolved.

@sdumitriu

This comment was marked as resolved.

@sdumitriu

This comment was marked as resolved.

@sdumitriu

This comment was marked as resolved.

@sdumitriu

This comment was marked as resolved.

@sdumitriu

This comment was marked as resolved.

@veronikaslc

This comment was marked as resolved.

Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't there have been a change that invalidates the data when saving is initiated?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do you mean the one removed here?

Co-authored-by: Marta Girdea <651980+marta-@users.noreply.github.com>
@marta-

This comment was marked as resolved.

@veronikaslc

This comment was marked as resolved.

@veronikaslc veronikaslc force-pushed the ui-dependency-update branch from 7b4c6a7 to 23971c2 Compare March 17, 2025 22:59
@veronikaslc veronikaslc force-pushed the ui-dependency-update branch from 17186fa to 6edf9f2 Compare March 18, 2025 20:05
@veronikaslc
Copy link
Contributor Author

The problem left to fix:

  • open the form (view mode) in two tabs
  • edit, change and save in one tab
  • edit in the second tab -> does it load the correct data?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file large PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants