Skip to content

Expose rrc_text via prefab - #43275

Closed
tomekzaw wants to merge 3 commits into
react:mainfrom
tomekzaw:patch-3
Closed

Expose rrc_text via prefab#43275
tomekzaw wants to merge 3 commits into
react:mainfrom
tomekzaw:patch-3

Conversation

@tomekzaw

Copy link
Copy Markdown
Contributor

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-botfacebook-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

analysis-bot commented Mar 4, 2024

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
androidhermesarm64-v8a18,016,657+0
androidhermesarmeabi-v7an/a--
androidhermesx86n/a--
androidhermesx86_64n/a--
androidjscarm64-v8a21,375,071-6
androidjscarmeabi-v7an/a--
androidjscx86n/a--
androidjscx86_64n/a--

Base commit: dc254ac
Branch: main

@cortinico

Copy link
Copy Markdown
Contributor

@tomekzaw can you rebase and fix the merge conflicts

@tomekzaw

Copy link
Copy Markdown
ContributorAuthor

@cortinico Done, conflicts resolved

@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@tomekzaw

Copy link
Copy Markdown
ContributorAuthor

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

@cortinico

Copy link
Copy Markdown
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-botfacebook-github-bot added the Merged This PR has been merged. label Mar 6, 2024
@facebook-github-bot

Copy link
Copy Markdown
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
@cipolleschicipolleschi mentioned this pull request Jul 22, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.MergedThis PR has been merged.p: Software MansionPartner: Software MansionPartnerShared with MetaApplied 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

@tomekzaw@analysis-bot@cortinico@facebook-github-bot