[FIX] Remove Typos in JSX, Add Unique Keys to props and Add id field in Forms

[samad-yar-khan]

• I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
• I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
• Lint and unit tests pass locally with my changes
• I have added tests that prove my fix is effective or that my feature works (if applicable)
• I have added necessary documentation (if applicable)
• Any dependent changes have been merged and published in downstream modules

Proposed changes (including videos or screenshots)

• I added unique key props to cater to different environments. id for Strapii and _id for mongoDB.

• Fixed Typos in JSX.

• Added id in searchBox Toggle to avoid warning : Issue

• Note : The only warning being displayed on console now are due to the issues persisting in the react-slick library. Reference

Issue(s)

Fixes : #58

Steps to test or reproduce

Mentioned in the Issue

Before :

2022-01-26.20-20-49_Trim.mp4

After :

2022-01-26.20-17-33_Trim.mp4

[Sing-Li]

What are the remaining 3 errors? Are they unfixable?

[samad-yar-khan]

@Sing-Li Unfortunately, the other three errors are due to the issue in the react-slick module. It has been closed but it still hasn't been solved

[Sing-Li]

@RonLek since it was your choice of component to use - do you think we should fork it and fix the problem?

[Sing-Li]

@samad-yar-khan please resolve the conflicts. We'll merge this after that.

[samad-yar-khan]

@samad-yar-khan please resolve the conflicts. We'll merge this after that.

@Sing-Li , will surely do. I tried implement a hack to resolve the issue with the slick library, but didn't really work out. Apart from that I am not getting any other warnings from this branch. I will try to solve the issue one more time and update you on this thread.

[lgtm-com]

This pull request introduces 1 alert when merging 3b5d998 into e5d3c71 - view on LGTM.com

new alerts:

• 1 for Unused variable, import, function or class

[debdutdeb]

forcing workflow run

[samad-yar-khan]

@samad-yar-khan please resolve the conflicts. We'll merge this after that.

@Sing-Li Resolved Errors and Rebased with the current master branch. It would be great if you could take a look 😄