Skip to content

Annotate throwable methods in ColorPropConverter, log bad Android resource lookup (#57239)#57239

Closed
Abbondanzo wants to merge 1 commit into
react:mainfrom
Abbondanzo:export-D108777172
Closed

Annotate throwable methods in ColorPropConverter, log bad Android resource lookup (#57239)#57239
Abbondanzo wants to merge 1 commit into
react:mainfrom
Abbondanzo:export-D108777172

Conversation

@Abbondanzo

@Abbondanzo Abbondanzo commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Summary:

Adds logcat logging when resource lookups fail if resource_paths is defined for a color prop, and annotates the ColorProp calls with the throwable class that they can raise

Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D108777172

@meta-codesync

meta-codesync Bot commented Jun 16, 2026

Copy link
Copy Markdown

@Abbondanzo has exported this pull request. If you are a Meta employee, you can view the originating Diff in D108777172.

@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 Jun 16, 2026
…ource lookup (react#57239)

Summary:

Adds logcat logging when resource lookups fail if `resource_paths` is defined for a color prop, and annotates the `ColorProp` calls with the throwable class that they can raise

Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D108777172
@meta-codesync meta-codesync Bot changed the title Annotate throwable methods in ColorPropConverter, log bad Android resource lookup Annotate throwable methods in ColorPropConverter, log bad Android resource lookup (#57239) Jun 17, 2026
@meta-codesync meta-codesync Bot closed this in 8a3549a Jun 17, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Jun 17, 2026
@meta-codesync

meta-codesync Bot commented Jun 17, 2026

Copy link
Copy Markdown

This pull request has been merged in 8a3549a.

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 Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant