React 18 migration

[Aveline-art]

Dependency

• Landing Page: Bottom Portion  #74
• Include a testing framework for react/js code #96

Overview

As a developer, we should utilize the newest technologies as it is more performant and offer enhanced features to support our work. For this issue, we will migrate from React 17 to 18.

Action Items

• Read the resources below to get updates on React18.
• Migrate over the React18, making code changes as needed. These changes are required:
  • Use the useId() hook to create keys for map rendering, as using index may cause memory leaks in the long term not recommended by docs
  • Update react-testing-library to v13+, since it will break after migration (see note)

Resources/Instructions

Resources
How to Upgrade to React 18
React v18.0
React 18 provides useId API for generating...

[Aveline-art]

Note to team: migrating to React18 will break our product on Internet Explorer (the browser). Please consider that before moving this issue forward.

Update April 28, 2022

Our usage statistics indicate no user uses IE from the start of the year up until now. Suffice to say, our users are using the latest tech :)

[sdimran]

@Aveline-art can you still add the necessary labels before we put this into the icebox -- sharad and i can then review it accordingly for the details. Thanks!

[Aveline-art]

There's probably no need to review it now since it is in the icebox and details are liable to change. Can I hold off on this until dependencies are done?

[Aveline-art]

Also, it seems to have been moved into new issue approval. Should I move it back? I'm not sure it is ready for approval yet.

[sdimran]

Hi Ava my apologies for the late response, I moved it into new issue approval because even if we are not working on this currently, it will be worked on down the line and we should ensure that the necessary details and labels (at the minimum the role and features, and an estimate size which could be changed later) are included for whoever does pick this up when the dependencies are completed. we can discuss in detail at the planning meeting tomorrow if needed as well