Skip to content

Conversation

@Gongreg
Copy link

@Gongreg Gongreg commented Aug 24, 2017

Motivation

Sometimes there is a need to set DatePickerIOS locale manually. For that I added optional prop locale. If user pass locale prop the DatePickerIOS will use it, otherwise it will use current phone locale.

Issues: #12405, mmazzarolo/react-native-modal-datetime-picker#31

Test Plan

Added location to RNTester DatePickerIOS screen. You can type locale and it will change to that one. For checking default locale I change phone language/region to other and it used the new locale.

@facebook-github-bot facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Aug 24, 2017
@pull-bot
Copy link

pull-bot commented Aug 24, 2017

Warnings
⚠️

📋 Release Notes - This PR appears to be missing Release Notes.

@facebook-github-bot label Needs more information

Attention: @shergin

Generated by 🚫 dangerJS

@Gongreg
Copy link
Author

Gongreg commented Aug 31, 2017

Hello,
Should I do something next about this PR? I see that CI is broken for all PR's.

@facebook-github-bot
Copy link
Contributor

@Gongreg I tried to find reviewers for this pull request and wanted to ping them to take another look. However, based on the blame information for the files in this pull request I couldn't find any reviewers. This sometimes happens when the files in the pull request are new or don't exist on master anymore. Is this pull request still relevant? If yes could you please rebase? In case you know who has context on this code feel free to mention them in a comment (one person is fine). Thanks for reading and hope you will continue contributing to the project.

@facebook-github-bot
Copy link
Contributor

@Gongreg I tried to find reviewers for this pull request and wanted to ping them to take another look. However, based on the blame information for the files in this pull request I couldn't find any reviewers. This sometimes happens when the files in the pull request are new or don't exist on master anymore. Is this pull request still relevant? If yes could you please rebase? In case you know who has context on this code feel free to mention them in a comment (one person is fine). Thanks for reading and hope you will continue contributing to the project.

@Gongreg
Copy link
Author

Gongreg commented Nov 20, 2017

#16639 already solved this. I will just close this one then..

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants