-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
[Standalone for NativeComponents] RCTSegmentedControl #23000
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code analysis results:
eslint
found some issues.
Libraries/Components/SegmentedControlIOS/RCTSegmentedControlNativeComponent.js
Outdated
Show resolved
Hide resolved
Libraries/Components/SegmentedControlIOS/RCTSegmentedControlNativeComponent.js
Outdated
Show resolved
Hide resolved
Generated by 🚫 dangerJS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TheSavior is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Thanks for contributing! |
Summary: Changelog: ---------- [iOS] [Changed] - follow facebook#22990, move the call to requireNativeComponent from `SegmentedControlIOS.ios.js` to `RCTSegmentedControlNativeComponent.js` Pull Request resolved: facebook#23000 Differential Revision: D13697168 Pulled By: TheSavior fbshipit-source-id: 83a66279d2fcd5a29d2ebc9cf5b5273947d96281
Changelog:
[iOS] [Changed] - follow #22990, move the call to requireNativeComponent from
SegmentedControlIOS.ios.js
toRCTSegmentedControlNativeComponent.js
Test Plan: