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

Expose rrc_text via prefab #43275

Closed
wants to merge 3 commits into from
Closed

Expose rrc_text via prefab #43275

wants to merge 3 commits into from

Conversation

tomekzaw
Copy link
Contributor

@tomekzaw tomekzaw commented Mar 1, 2024

Summary:

The rrc_text was not exposed via prefab. I'm adding it to make possible for react-native-live-markdown to integrate on top of React Native via prefab. Based on #36166.

Changelog:

[ANDROID] [CHANGED] - Expose rrc_text via prefab.

Test Plan:

@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: Software Mansion Partner: Software Mansion Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Mar 1, 2024
@analysis-bot
Copy link

analysis-bot commented Mar 4, 2024

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 18,016,657 +0
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 21,375,071 -6
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: dc254ac
Branch: main

@cortinico
Copy link
Contributor

@tomekzaw can you rebase and fix the merge conflicts

@tomekzaw
Copy link
Contributor Author

tomekzaw commented Mar 4, 2024

@cortinico Done, conflicts resolved

@facebook-github-bot
Copy link
Contributor

@cortinico has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@tomekzaw
Copy link
Contributor Author

tomekzaw commented Mar 6, 2024

@cortinico Could you please check why Facebook Internal - Builds & Tests workflow is failing?

@cortinico
Copy link
Contributor

@cortinico Could you please check why Facebook Internal - Builds & Tests workflow is failing?

Yup is some internal flakyness. I've triggered a land, should be merged in a couple of hours at most

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Mar 6, 2024
@facebook-github-bot
Copy link
Contributor

@cortinico merged this pull request in 1c69100.

huntie pushed a commit that referenced this pull request Mar 11, 2024
Summary:
The `rrc_text` was not exposed via prefab. I'm adding it to make possible for react-native-live-markdown to integrate on top of React Native via prefab. Based on #36166.

## Changelog:

<!-- Help reviewers and the release process by writing your own changelog entry.

Pick one each for the category and type tags:

[ANDROID|GENERAL|IOS|INTERNAL] [BREAKING|ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY] - Message

For more details, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests
-->

[ANDROID] [CHANGED] - Expose `rrc_text` via prefab.

Pull Request resolved: #43275

Reviewed By: cipolleschi

Differential Revision: D54536468

Pulled By: cortinico

fbshipit-source-id: 8c4ef983467bfc46930f10bf7bd95761c2d11788
This was referenced Jun 28, 2024
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. Merged This PR has been merged. p: Software Mansion Partner: Software Mansion Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants