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

feat: Add inset logical properties #12

Merged

Conversation

gabrieldonadel
Copy link
Contributor

This PR updates the DeprecatedLayoutPropTypes to include the inset logical properties introduced by facebook/react-native@9669c10.

@NickGerleman
Copy link
Contributor

TIL I have permissions to this repo

@NickGerleman NickGerleman merged commit 57e71fd into facebook:main Jan 6, 2023
@NickGerleman
Copy link
Contributor

facebook/react-native#35572 is landing now, and I'm following up with the internally running screenshot tests. @gabrieldonadel you should be safe now to add the prop-types here as well.

@NickGerleman
Copy link
Contributor

I'm going to add this repo to the list with generate notifications for our oncall on PR as well, since it's low-traffic enough that I am not sure folks are religiously checking it.

@NickGerleman
Copy link
Contributor

I'm going to add this repo to the list with generate notifications for our oncall on PR as well, since it's low-traffic enough that I am not sure folks are religiously checking it.

Done. Future PRs to this repo will generate Workplace notifications internally now.

@gabrieldonadel gabrieldonadel deleted the feat/add-inset-logical-properties branch January 6, 2023 12:07
@gabrieldonadel
Copy link
Contributor Author

Sounds good, thanks @NickGerleman

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants