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

chore: Add iOS 17 textContentType types #23

Merged

Conversation

robwalkerco
Copy link
Contributor

This prop type change is required to support the new textContentType and autoComplete prop values that are being added in facebook/react-native#38354

If/when this PR is merged and released, the associated react-native PR will need to have its deprecated-react-native-prop-types dependency updated to prevent prop type warnings.

@robwalkerco robwalkerco changed the title fix: add ios17 types chore: Add iOS 17 textContentType types Jul 14, 2023
@philIip philIip merged commit 9ed6f11 into facebook:main Jul 17, 2023
@robwalkerco robwalkerco deleted the fix/add-ios17-auto-complete-types branch July 17, 2023 22:13
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Jul 21, 2023
Summary:
While setting up a credit card form in RN, I discovered that iOS 17 supports a number of new content types ([`UITextContentType` docs](https://developer.apple.com/documentation/uikit/uitextcontenttype?language=objc)). In the docs these are marked with the `Beta` flag.

Setting up the new content types is relatively straightforward, but a change is required in https://github.com/facebook/react-native-deprecated-modules to update the `TextInput` prop types. ~~I will open a PR in that repo shortly.~~ I have [opened a PR](facebook/react-native-deprecated-modules#23) to update the prop types. ~~Once that PR is merged, a version bump for that dependency will need to be added to this PR.~~ The PR is merged and the dependency in this PR has been updated.

## Changelog:

<!-- Help reviewers and the release process by writing your own changelog entry.

Pick one each for the category and type tags:

[IOS] [ADDED] - Added support for iOS 17+ text content types

For more details, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests

Pull Request resolved: #38354

Test Plan: The `rn-tester` app builds and runs successfully. I have added a few new examples of inputs using the new text content types.

Reviewed By: javache

Differential Revision: D47554161

Pulled By: philIip

fbshipit-source-id: 8d4414dc6229063f81164f2d8727921c8294c92e
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