Convert legacy React class components to function components

[lindapaiste]

Increasing Access

It keeps the codebase up to date and makes it easier for new contributors.

Feature enhancement details

React.Component is considered a legacy API and is not recommended. We should convert all class components to function components which use React hooks.

---

This is mainly a "Progress Report" for myself because with so many unmerged PRs sometimes I forget what I've already worked on!

Here are the class components which remain the codebase, and the PRs to convert them:

• EmailVerificationView
• App Convert App to a function component. #2308
• Overlay Convert Overlay to a function component, shares logic with Modal #2309
• AddToCollectionList Fix #2351 & cleanup AddToCollectionList component #2353
• AddToCollectionSketchList Convert AddToCollectionSketchList to a function component #2364
• ConsoleInput
• CopyableInput Convert CopyableInput to a function component #2360
• Editor -- this one is too annoying/difficult due to the potential for stale closures. It will likely wait until upgrading to codemirror v6, when the component needs a lot of changes anyways.
• FileNode
• Toolbar Convert Toolbar to a function component #2352
• Preferences Convert Preferences into a function component #2335
• Searchbar
• IDEView done by @dewanshDT, not submitted yet
• APIKeyForm Convert APIKeyForm to a function component and connect to Redux #2307
• Collection Convert Collection to a function component #2366
• CollectionView Delete unused code in CollectionView #2302
• DashboardView
• Tables -- should be addressed together (see Create generic Table component #1442)
  • SketchListRowBase
  • SketchList Common Table component (part 1) #2376
  • AssetListRowBase
  • AssetList Common Table component (part 1) #2376
  • CollectionListRowBase
  • CollectionList Common Table component (part 1) #2376

[vijayabhaskar-ev]

Convert EmailVerificationView to function component #2401

[KSSaiTeja]

i want to work to work to convert the collectionListRowbase component and make a PR. can i do it?

[shauryag2002]

Hi @lindapaiste , It seems like you are working on this issue quite long time,
if not able to do it !
Can @lindapaiste assign it to me?

[lindapaiste]

Thank you everyone for all of your work on this! It looks like all components have been covered in pending PRs.

We do not need any more new PRs for this issue.

[KSSaiTeja]

@lindapaiste could you please take my PR into consideration and review it

[up1512001]

Can I work on DashboardView if anyone didn't?

[thakkarnetram]

Ref #2447 .
I have updated the Assets List to a functional component

[lindapaiste]

Normally I would not close an issue until all of the PRs are merged. But I'm going to close this so y'all will stop submitting unnecessary PRs.

[KSSaiTeja]

@lindapaiste what about my PR. will it be considered or closed? as a beginner i didn't understand your point.. please tell me

[thakkarnetram]

@lindapaiste what about my PR. will it be considered or closed? as a beginner i didn't understand your point.. please tell me

I didn't understand either

[umangutkarsh]

Hi @lindapaiste , I'd like to contribute to this issue, if this issue is still open.