Skip to content

Make react/renderer/imagemanager public (#58438) - #58438

Closed
j-piasecki wants to merge 2 commits into
react:mainfrom
j-piasecki:export-D119477737
Closed

Make react/renderer/imagemanager public (#58438)#58438
j-piasecki wants to merge 2 commits into
react:mainfrom
j-piasecki:export-D119477737

Conversation

@j-piasecki

@j-piasecki j-piasecki commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary:

Classifies react/renderer/imagemanager:imagemanager as a public target under the C++ stable API three-tier visibility model. Replaces the FrameworksGuard.h include in the module's 7 exported headers with <react/cxxstableapi/UmbrellaGuard.h> and introduces the module umbrella React/ImageManager.h, wiring the umbrella into BUCK, CMake, CocoaPods, the iOS prebuild header config and the Android prefab export.

Changelog:
[General][Added] - Added umbrella header for the react/renderer/imagemanager module

Reviewed By: cortinico

Differential Revision: D119477737

@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 Sep 10, 2026
@facebook-github-tools facebook-github-tools Bot added p: Software Mansion Partner: Software Mansion Partner p: Facebook Partner: Facebook labels Sep 10, 2026
@meta-codesync

meta-codesync Bot commented Sep 10, 2026

Copy link
Copy Markdown

@j-piasecki has exported this pull request. If you are a Meta employee, you can view the originating Diff in D119477737.

j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Sep 10, 2026
Summary:
Pull Request resolved: react#58438

Classifies `react/renderer/imagemanager:imagemanager` as a public target under the C++ stable API three-tier visibility model. Replaces the `FrameworksGuard.h` include in the module's 7 exported headers with `<react/cxxstableapi/UmbrellaGuard.h>` and introduces the module umbrella `React/ImageManager.h`, wiring the umbrella into BUCK, CMake, CocoaPods, the iOS prebuild header config and the Android prefab export.

Changelog:
[General][Added] - Added umbrella header for the `react/renderer/imagemanager` module

Differential Revision: D119477737
@meta-codesync meta-codesync Bot changed the title Make react/renderer/imagemanager public Make react/renderer/imagemanager public (#58438) Sep 10, 2026

@cortinico cortinico left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review automatically exported from Phabricator review in Meta.

Jakub Piasecki and others added 2 commits September 10, 2026 23:14
Summary:
Classifies `react/renderer/components/image:image` as a public target under the C++ stable API three-tier visibility model. Replaces the `FrameworksGuard.h` include in the module's 6 exported headers with `<react/cxxstableapi/UmbrellaGuard.h>` and introduces the module umbrella `React/Image.h`, wiring the umbrella into BUCK, CMake, CocoaPods, the iOS prebuild header config and the Android prefab export.

Changelog:
[General][Added] - Added umbrella header for the `react/renderer/components/image` module

Differential Revision: D119477736
Summary:
Pull Request resolved: react#58438

Classifies `react/renderer/imagemanager:imagemanager` as a public target under the C++ stable API three-tier visibility model. Replaces the `FrameworksGuard.h` include in the module's 7 exported headers with `<react/cxxstableapi/UmbrellaGuard.h>` and introduces the module umbrella `React/ImageManager.h`, wiring the umbrella into BUCK, CMake, CocoaPods, the iOS prebuild header config and the Android prefab export.

Changelog:
[General][Added] - Added umbrella header for the `react/renderer/imagemanager` module

Reviewed By: cortinico

Differential Revision: D119477737
@meta-codesync meta-codesync Bot closed this in be53e3a Sep 11, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Sep 11, 2026
@meta-codesync

meta-codesync Bot commented Sep 11, 2026

Copy link
Copy Markdown

This pull request has been merged in be53e3a.

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. meta-exported p: Facebook Partner: Facebook p: Software Mansion Partner: Software Mansion Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants