Page MenuHomePhabricator

Upgrade to Moment.js 2.6.0
Closed, ResolvedPublic

Description

Please upgrade Moment.js to version 2.6.0, which has many bug fixes and several new features. In particular, Moment.js was using an incorrect language code for Vietnamese (vn instead of vi); upgrading should fix the issue in which Hovercards were showing timestamps like “Sửa đổi lần cuối 9 days ago”.

As a side note, it needs to be easier for MediaWiki’s very thorough translators to find out that timestamps are being translated in an obscure third-party project on GitHub.


Version: unspecified
Severity: enhancement
URL: https://github.com/moment/moment/pull/1556

Details

Reference
bz63873

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 3:20 AM
bzimport set Reference to bz63873.
bzimport added a subscriber: Unknown Object (MLST).

(In reply to Minh Nguyễn from comment #0)

As a side note, it needs to be easier for MediaWiki’s very thorough
translators to find out that timestamps are being translated in an obscure
third-party project on GitHub.

Ugh. Yet another one? :< Please file a separate bug for that.

(In reply to Nemo from comment #1)

(In reply to Minh Nguyễn from comment #0)

As a side note, it needs to be easier for MediaWiki’s very thorough
translators to find out that timestamps are being translated in an obscure
third-party project on GitHub.

Ugh. Yet another one? :< Please file a separate bug for that.

Prateek, you may want to see [[mw:Extension:CLDR]] - I believe that's what everything else in MediaWiki uses, for timestamps. (Obligatory IANAD disclaimer :)

moment.js is part of core, so changing component.

(In reply to Quiddity from comment #2)

Prateek, you may want to see [[mw:Extension:CLDR]] - I believe that's what
everything else in MediaWiki uses, for timestamps. (Obligatory IANAD
disclaimer :)

Sadly there is no way to access CLDR in JS, so moment.js was merged to make up for that.

Changelog for 2.6.0 is at https://gist.github.com/ichernev/10544682 , none of it seems to be a breaking change.

Change 126236 had a related patch set uploaded by Prtksxna:
Update moment.js to 2.6.0

https://gerrit.wikimedia.org/r/126236

Change 126236 had a related patch set uploaded by Siebrand:
Update moment.js to 2.6.0

https://gerrit.wikimedia.org/r/126236

Change 126236 merged by jenkins-bot:
Update moment.js to 2.6.0

https://gerrit.wikimedia.org/r/126236