-
Notifications
You must be signed in to change notification settings - Fork 227
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
(fix) O3-3871 Upgrade react-to-print library to v3 beta to resolve TypeError #1979
base: main
Are you sure you want to change the base?
(fix) O3-3871 Upgrade react-to-print library to v3 beta to resolve TypeError #1979
Conversation
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.
Thank you @WodPachua.
Would also update parameters to useReactToPrint
in all its usages within this repo?
Is this the update you had proposed? @jnsereko |
4d4febc
to
8e27da5
Compare
react: ^15.0.0 || ^16.0.0 || ^17.0.0 || ^18.0.0 | ||
react-dom: ^15.0.0 || ^16.0.0 || ^17.0.0 || ^18.0.0 | ||
checksum: 10/cc4e438b5b7e5c67a73ca3d87ff0f1b5a550a7e6c1a1911cded14c9591b286efd82029bd0447e016b0b404f274d546a108e760fd7867e6b73342dba450282a29 | ||
react: ^16.8.0 || ^17.0.0 || ^18.0.0 || ~19 |
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.
@WodPachua was this neccessary?
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.
Isn't this auto-generated to reflect the dependency(react-to-print version) change?
Depends on and is blocked by #1934. |
Requirements
Summary
Upgraded the react-to-print library to version 3 beta to resolve a TypeError encountered during the first print attempt in the development environment. This issue was not observed in the production environment
Screenshots
Related Issue
O3-3871
Other