-
Notifications
You must be signed in to change notification settings - Fork 167
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
Flow type support for data-* props #71
Comments
While we wait for the Flow team to prioritise and implement the This will avoid the need for patch files when we try to maintain a list of props used internally. We can also shadow this file with a |
Allow arbitrary data-* props at runtime. On web, the check is only performed during development. To workaround Flow's lack of support for typing arbitary data-* props, a separate type is created just for data-* props which can be used to maintain an app's list of supported data-* props. Ref #71 Close #75
Allow arbitrary data-* props at runtime. On web, the check is only performed during development. To workaround Flow's lack of support for typing arbitary data-* props, a separate type is created just for data-* props which can be used to maintain an app's list of supported data-* props. Ref #71 Close #75
Allow arbitrary data-* props at runtime. On web, the check is only performed during development. To workaround Flow's lack of support for typing arbitary data-* props, a separate type is created just for data-* props which can be used to maintain an app's list of supported data-* props. Ref #71 Close #75
Allow arbitrary data-* props at runtime. On web, the check is only performed during development. To workaround Flow's lack of support for typing arbitary data-* props, a separate type is created just for data-* props which can be used to maintain an app's list of supported data-* props. Ref #71 Close #75
Describe the feature request
Flow doesn't have support for typing arbitrary
data-*
props. This requires RSD to add Meta-specificdata-*
props on an as-needed basis, and each time it is considered a "blocker" by the team migrating components to RSD. This creates incentives to either 1) patch internally generated RSD syncs with new Flow types (complicates syncs, can cause regressions), or 2) add Meta-specific props to the OSS project code (doesn't scale to other users).cc @jbrown215 who had ideas on how to support this from the Flow side
The text was updated successfully, but these errors were encountered: