Bootstrap v5, React bootstrap v2, React 18, Redux and Misc. Npm Packages Upgrade #775
Conversation
There was a problem hiding this comment.
Finally got around to looking at this. Code looks good aside from one minor detail I mentioned that might be worth backtracking on. I haven't been able to test this locally, however, as my CGAP local env is giving me a bit of trouble at the moment, and I haven't had time to fix it. Could you deploy this to wolf sometime soon and let us know? There are a few final things I want to double check and then this can be merged.
| // It appears that Cypress injects a prop into the div on the client side, which doesn't exist on the server side, leading to hydration issues. | ||
| // This suppression is temporary and should be removed once the issue is resolved. | ||
| // https://github.com/cypress-io/cypress/issues/27204 |
There was a problem hiding this comment.
Looks like that thread has introduced a new solution to this problem. Might be worth revisiting this and see if this can be removed/replaced with a more permanent solution before merge.
|
@Bianca-Morris no worries, thanks for the review. It is recently available in cgap-wolf. AFAIK, SPC's release was changed from beta to production. Anyway, we'll redeploy it and keep you posted.
|
|
@Bianca-Morris FYI, recent version is available in https://cgap-wolf.hms.harvard.edu |
This pull request primarily addresses the transition from React 17 to React 18, while also including updates for Redux, Auth0, and other third-party integrations heavily used in the portal, ensuring compatibility with React 18.
Ticket: Trello 1 & Trello 2
Related PR: SPC
Updates
Additionally, this pull request implements the migration to Bootstrap v5 and updates to React-Bootstrap v2, ensuring that styles and components are fully compatible with the latest standards.