Replace react-hot-loader
Description
Environment
None
Potential Workaround
None
is blocked by
relates to
Checklist
hideTestRail: Results
Activity
Show:
Zak Burke September 19, 2022 at 3:06 PM
No concerns.
Khalilah Gambrell September 19, 2022 at 2:52 PM
Hey
Should we do it for ui-calendar since Team Bama has a refactored Calendar app?
I am unsure how many libraries actually have the Codex app enabled.
So I guess, I propose closing this issue with a note that it was not done for ui-calendar and ui-search. Any concerns?
Zak Burke September 19, 2022 at 11:27 AM
, alas, no; it was never removed from ui-calendar or ui-search. The change is small and inconsequential. I can issue the PRs and make releases if you want.
Khalilah Gambrell September 17, 2022 at 1:40 PM
and was the work completed in Morning Glory? If so, can we close this story?
Marc Johnson March 30, 2022 at 8:57 AM
any work that's already done was done post-Lotus so it's all targeted at Morning Glory
Thank you for confirming that
Done
Details
Details
Assignee
Unassigned
UnassignedReporter
Michal Kuklis
Michal KuklisPriority
P3Development Team
Prokopovych
Release
Morning Glory (R2 2022)
TestRail: Cases
Open TestRail: Cases
TestRail: Runs
Open TestRail: Runs
Created January 11, 2021 at 7:14 PM
Updated September 19, 2022 at 4:13 PM
Resolved September 19, 2022 at 4:13 PM
TestRail: Cases
TestRail: Runs
react-hot-loader has been deprecated and it looks like it's causing some issues with to lazy loading in react 17.
We should review:
https://github.com/pmmmwh/react-refresh-webpack-plugin
react-hot-loader hasn't been maintained since Jun 1, 2021. There are 116 open issues and 246 open pull requests as of March 22, 2022: https://github.com/gaearon/react-hot-loader
react-hot-loader has security vulnerabilities: https://nvd.nist.gov/vuln/detail/CVE-2020-7598 , https://nvd.nist.gov/vuln/detail/CVE-2021-44906
Steps to Fix:
The react-hot-loader maintainers say: "Please remove React-Hot-Loader": https://github.com/gaearon/react-hot-loader#moving-towards-next-step
The react-hot-loader maintainers advise to replace it by React Hot Refresh: https://github.com/facebook/react/issues/16604