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.

Sign up for GitHub

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

Jump to bottom

PoC: Rough example of adopting createContext #3751

Draft

patlm wants to merge 3 commits into box:master from patlm:remove-legacy-context-apis-usage

Contributor

patlm commented Nov 20, 2024

Verified that the migration works on the all files page but not much else beyond basic proof-of-concept
Actual PR should verify tests are passing and component functionality is the same


          Example

bcf89b6

patlm added the do-not-merge label

patlm and others added 2 commits

November 20, 2024 15:52


          Delete src/components/form-elements/form/Form.js

e107e49

Removing from example to avoid confusion


          Remove form changes to avoid confusion

f6d16cc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels