Skip to content

Move ShadowNodePair to Differentiator#46590

Closed
javache wants to merge 2 commits intofacebook:mainfrom
javache:export-D63148522
Closed

Move ShadowNodePair to Differentiator#46590
javache wants to merge 2 commits intofacebook:mainfrom
javache:export-D63148522

Conversation

@javache
Copy link
Copy Markdown
Member

@javache javache commented Sep 20, 2024

Summary:
This API should not be part of the ShadowView header, since the class is only used by Differentiator. The ShadowView header should contain public API's that the mounting layer can consume.

Changelog: [Internal]

Differential Revision: D63148522

@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 Sep 20, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

javache and others added 2 commits September 23, 2024 02:44
Differential Revision: D63099721
Summary:
Pull Request resolved: facebook#46590

This API should not be part of the ShadowView header, since the class is only used by Differentiator. The ShadowView header should contain public API's that the mounting layer can consume.

Changelog: [Internal]

Reviewed By: sammy-SC

Differential Revision: D63148522
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

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

This pull request has been merged in a6fcad7.

@javache javache deleted the export-D63148522 branch September 26, 2024 16:28
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