-
Notifications
You must be signed in to change notification settings - Fork 3.7k
Migrate off of findNodeHandle #11635
Copy link
Copy link
Closed
Labels
DailyKSv2KSv2EngineeringInternalRequires API changes or must be handled by Expensify staffRequires API changes or must be handled by Expensify staffReviewingHas a PR in reviewHas a PR in reviewTask
Description
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Problem
According to the official docs, we cannot turn on Fabric (RN new architecture) without first migrating away all uses of findNodeHandle
Solution
Migrate away from usages of findNodeHandle. We currently only have one in BaseOptionSelector here
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
DailyKSv2KSv2EngineeringInternalRequires API changes or must be handled by Expensify staffRequires API changes or must be handled by Expensify staffReviewingHas a PR in reviewHas a PR in reviewTask