Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert ./components/suggestion/search_date_suggestion/search_date_suggestion.tsx from Class Component to Function Component #26695

Closed
larkox opened this issue Apr 5, 2024 · 3 comments · Fixed by #26708
Assignees
Labels
Difficulty/1:Easy Easy ticket Good First Issue Suitable for first-time contributors Help Wanted Community help wanted Tech/TypeScript

Comments

@larkox
Copy link
Contributor

larkox commented Apr 5, 2024

We want to move most of our codebase from Class Components to Function Components. This decision is based on React recommendations.

The task is to update the component to be a functional component, without any difference in functionality. Also, all related tests that fail may need refactoring.

Here are a few examples of the required changes:
#24187
#24210
#24183
#24181

If the tests do not work because they rely on the behavior of Class Components they also have to be migrated to functionaly test the same behavior. We prefer the use of react-testing-library for new tests, but it is not required.

When submitting the PR, please make sure to identify where and when the component shows, for QA to properly test the changes.


If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA LINK: MM-57727

@larkox larkox added Difficulty/1:Easy Easy ticket Good First Issue Suitable for first-time contributors Help Wanted Community help wanted Tech/TypeScript Up For Grabs labels Apr 5, 2024
@Syed-Ali-Abbas-Zaidi
Copy link
Contributor

Please Assign this to me.

@MeHow25
Copy link
Contributor

MeHow25 commented Apr 7, 2024

Hi @Syed-Ali-Abbas-Zaidi ! I saw that you have many issues assigned so I wanted to help you by fixing this one. Please, feel free to review my PR.

@larkox
Copy link
Contributor Author

larkox commented Apr 8, 2024

@Syed-Ali-Abbas-Zaidi Since @MeHow25 already created a PR and you have 19 other tickets to work on, I will assign this one to @MeHow25
I hope you don't mind.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Difficulty/1:Easy Easy ticket Good First Issue Suitable for first-time contributors Help Wanted Community help wanted Tech/TypeScript
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants