Convert ./components/integrations/add_incoming_webhook/add_incoming_webhook.tsx from Class Component to Function Component #26694
Assignees
Labels
Difficulty/1:Easy
Easy ticket
Good First Issue
Suitable for first-time contributors
Help Wanted
Community help wanted
Tech/TypeScript
Comments
larkox
added
Difficulty/1:Easy
|
Please Assign this to me. |
|
@Syed-Ali-Abbas-Zaidi Since you have a few PRs that need addressing, and you haven't been able to address this one yet, I will open it up for up for grabs. |
|
I can work on it |
|
@MeHow25 All yours! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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-libraryfor 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-57726
The text was updated successfully, but these errors were encountered: