Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
[Intl] Document timezone support#6209
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
c960657 commentedFeb 1, 2016
| Q | A |
|---|---|
| Doc fix? | no |
| New docs? | yes (symfony/symfony#17636) |
| Applies to | 3.0+ |
| Fixed tickets | N/A |
wouterj commentedFeb 6, 2016
👍 After your code PR has been merged, a |
components/intl.rst Outdated
| \Locale::setDefault('en'); | ||
| $locales = Intl::getTimezoneBundle()->getTimezoneNames(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
$locales variable doesn't store locales but time zone names. Same few lines below.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Fixed.
| Timezones | ||
| ~~~~~~~~~ | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
version added ?
c960657 commentedJul 13, 2017
Closing becausesymfony/symfony#17636 was rejected. |