Skip to content

Conversation

@lenaic
Copy link

@lenaic lenaic commented Nov 8, 2025

Summary:
Even though the prop is not used on the Android view manager, adding prop diffing will reduce false error reports coming from the Props 2.0 runtime validator.

Changelog: [Internal]

Differential Revision: D86577393

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 8, 2025
@meta-codesync
Copy link

meta-codesync bot commented Nov 8, 2025

@lenaic has exported this pull request. If you are a Meta employee, you can view the originating Diff in D86577393.

lenaic pushed a commit to lenaic/react-native-macos that referenced this pull request Nov 10, 2025
Summary:

Even though the prop is not used on the Android view manager, adding prop diffing will reduce false error reports coming from the Props 2.0 runtime validator.

Changelog: [Internal]

Reviewed By: christophpurrer

Differential Revision: D86577393
Nick Lefever added 2 commits November 10, 2025 10:05
Summary:

With `opacity` being initialized to NaN for text attributes, not having the NaN check would mean adding the `opacity` prop to the Props 2.0 diff output even when it was unchanged and still set to NaN

Changelog: [Internal]

Reviewed By: christophpurrer

Differential Revision: D86577389
Summary:

Even though the prop is not used on the Android view manager, adding prop diffing will reduce false error reports coming from the Props 2.0 runtime validator.

Changelog: [Internal]

Reviewed By: christophpurrer

Differential Revision: D86577393
@meta-codesync meta-codesync bot closed this in 4f49081 Nov 11, 2025
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Nov 11, 2025
@meta-codesync
Copy link

meta-codesync bot commented Nov 11, 2025

This pull request has been merged in 4f49081.

douglowder pushed a commit to react-native-tvos/react-native-tvos that referenced this pull request Nov 15, 2025
Summary:
Pull Request resolved: facebook/react-native#54463

Even though the prop is not used on the Android view manager, adding prop diffing will reduce false error reports coming from the Props 2.0 runtime validator.

Changelog: [Internal]

Reviewed By: christophpurrer

Differential Revision: D86577393

fbshipit-source-id: 748ca90a555eb1f98be551540ad2d54aba08cf25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants