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

Updated moment.d.ts #3244

Closed
wants to merge 2 commits into from
Closed

Updated moment.d.ts #3244

wants to merge 2 commits into from

Conversation

jvanoostveen
Copy link
Contributor

The current version (released version) is missing a definition for updateLocale in the moment namespace, the dev branch version also has an issue fixed in this pr.

According to the documentation, the locale can also be updated on the global moment, ie. moment.updateLocale(...).
@ichernev
Copy link
Contributor

its merged from #3135

@ichernev ichernev closed this Jun 16, 2016
@jvanoostveen
Copy link
Contributor Author

jvanoostveen commented Jun 16, 2016

The changes in #3135 are not the same changes.

Currently, without the changes from this PR, the following is not possible:

import * as moment from 'moment';

moment.updateLocale('nl', {...});

As the updateLocale method in the definition file is only for the moment instance, i.e. moment().updateLocale('nl', {...});.

Also the first commit contains a bugfix needed otherwise it would not compile at all.

@ichernev ichernev reopened this Jun 16, 2016
@ichernev
Copy link
Contributor

Merged in a2c9e3e

@ichernev ichernev closed this Jun 20, 2016
ichernev added a commit that referenced this pull request Jun 20, 2016
@mattjohnsonpint mattjohnsonpint added this to the 2.14.0 milestone Jul 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants