-
Notifications
You must be signed in to change notification settings - Fork 25.1k
Convert components off of createReactClass #21581
Copy link
Copy link
Closed
Labels
Good first issueInterested in collaborating? Take a stab at fixing one of these issues.Interested in collaborating? Take a stab at fixing one of these issues.Help Wanted
Issues ideal for external contributors.Issues ideal for external contributors.JavaScriptResolution: LockedThis issue was locked by the bot.This issue was locked by the bot.
Metadata
Metadata
Assignees
Labels
Good first issueInterested in collaborating? Take a stab at fixing one of these issues.Interested in collaborating? Take a stab at fixing one of these issues.Help Wanted
Issues ideal for external contributors.Issues ideal for external contributors.JavaScriptResolution: LockedThis issue was locked by the bot.This issue was locked by the bot.
Type
Fields
Give feedbackNo fields configured for issues without a type.
As part of our goal to modernize and type our core components, we need to migrate off of
createReactClass. This task is to track the work to removecreateReactClassfrom our codebase.We can only convert off of
createReactClassif the components don't use any mixins. Step 1 and Step 2 were removing a few of the mixins. There are still a few mixins left and we will need to completely finish that before we can complete this issue.However, we can at least get started by removing
createReactClassfrom the components that currently don't use any mixins. The following is that list:I got this list by running
grep -L "mixins:" $(grep -l -r createReactClass .). None of these files seem to be convertible by the codemod provided by the react team.I'm providing less instructions on this one because I think there are a lot of posts online about how to convert off of
createReactClass.