Skip to content
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

BridgelessUIManager: Align platform-only methods w/ PaperUIManager #41997

Closed
wants to merge 1 commit into from

Conversation

RSNara
Copy link
Contributor

@RSNara RSNara commented Dec 19, 2023

Summary:
Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Differential Revision: D52012876

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Dec 19, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52012876

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52012876

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52012876

RSNara added a commit to RSNara/react-native that referenced this pull request Dec 26, 2023
…acebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Reviewed By: sammy-SC

Differential Revision: D52012876
@RSNara RSNara force-pushed the export-D52012876 branch 2 times, most recently from b77c800 to 6c2b506 Compare December 26, 2023 18:14
RSNara added a commit to RSNara/react-native that referenced this pull request Dec 26, 2023
…acebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Reviewed By: sammy-SC

Differential Revision: D52012876
RSNara added a commit to RSNara/react-native that referenced this pull request Dec 26, 2023
…acebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Reviewed By: sammy-SC

Differential Revision: D52012876
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52012876

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52012876

RSNara added a commit to RSNara/react-native that referenced this pull request Dec 26, 2023
…acebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Reviewed By: sammy-SC

Differential Revision: D52012876
RSNara added a commit to RSNara/react-native that referenced this pull request Dec 26, 2023
…acebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Reviewed By: sammy-SC

Differential Revision: D52012876
RSNara added a commit to RSNara/react-native that referenced this pull request Dec 26, 2023
…acebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Reviewed By: sammy-SC

Differential Revision: D52012876
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52012876

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52012876

RSNara added a commit to RSNara/react-native that referenced this pull request Dec 26, 2023
…acebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52012876

RSNara added a commit to RSNara/react-native that referenced this pull request Dec 26, 2023
…acebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52012876

RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: 678a21a0f52ba87d9d0d8843dc414dff5c672cb3
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: 1ad8327937d173f7e9fdde345102ad3f3a630a69
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: 3f7aff6d73c5b63bd1d905c2b527544e7b0c03b4
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: f6c2e521e49bfeb6a21b952bff881eb843a931c0
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: f45529fa33842e0571507c54aed381a9d9a47669
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…w/ PaperUIManager (facebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: c58402b443004e71ebae9d3c214a3a13d9dcb1b5
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: 88fb5de9aa9b0c4e7294f12f7d82d18233b98875
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: d361b1324855ec782664df7c4eda2d8ec60c1e1c
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: db5d3d0a5947e2379b143f190234d5df270dbaa7
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: c813be1240331f53d3568dc2e30e0e22798c5320
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: 43a662ed0a4a56a623b5a83c0862da6da0e0ee3c
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: dab04af2b14cf44723ef1bf79afed5ddc2b96b3b
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: 2fc765737234e4e2f5264dd64efc56b5e970b849
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: 4f8925c126fdff7cd431c76d7f0998c2a3625adb
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: 8a38ef0570bcbdf8858a22837f2d75671c86bb7c
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: d126b34271b9ccabdb395e26a85d6068fb43247f
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: 2c7802df97522b56633e707290c3591c2a1462c6
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 9, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: f40cf84281b03cacdc431e264ec6084eb7c90171
…acebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 10, 2024
…Manager (facebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876
RSNara added a commit to RSNara/react-native that referenced this pull request Jan 10, 2024
…Manager (facebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52012876

RSNara added a commit to RSNara/react-native that referenced this pull request Jan 10, 2024
…Manager (facebook#41997)

Summary:

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876
Copy link

This pull request was successfully merged by @RSNara in 4ed4ea1.

When will my fix make it into a release? | Upcoming Releases

@github-actions github-actions bot added the Merged This PR has been merged. label Jan 10, 2024
gokul1099 pushed a commit to gokul1099/react-native-visionos that referenced this pull request Jan 17, 2024
…acebook#41997)

Summary:
Pull Request resolved: facebook#41997

Many methods on PaperUIMangaer are iOS only.

Many methods on PaperUIManager are Android  only.

This diff makes sure that BridgelessUIManager only exports Android methods on Android, and iOS methods on iOS.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D52012876

fbshipit-source-id: 6527048083eae93577a58d4b77f0645fab84217f
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. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants