Move remaining DateTimeFormat uses to Intl/async

RESOLVED FIXED in FxOS-S7 (18Sep)

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: gandalf, Assigned: gandalf)

Tracking

unspecified
FxOS-S7 (18Sep)
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [systemsfe])

Attachments

(1 attachment)

There's still a bunch of date time formatting placed where System uses l10n_date's functions.

For all formattings we should use Intl, and for prettyDate we should use an async version.
(Assignee)

Updated

3 years ago
Assignee: nobody → gandalf
Blocks: 1170963
Created attachment 8659017 [details] [review]
[gaia] zbraniecki:1203348-system-intl > mozilla-b2g:master
Whiteboard: [systemsfe]
(Assignee)

Comment 2

3 years ago
Comment on attachment 8659017 [details] [review]
[gaia] zbraniecki:1203348-system-intl > mozilla-b2g:master

Alberto, can you review this or redirect to someone who can?
Attachment #8659017 - Flags: review?(apastor)
Comment on attachment 8659017 [details] [review]
[gaia] zbraniecki:1203348-system-intl > mozilla-b2g:master

Just a quick comment in GH about a possible remaining file, but the code looks good to me. Thanks for fixing this!
Attachment #8659017 - Flags: review?(apastor) → review+
(Assignee)

Updated

3 years ago
Keywords: checkin-needed
https://github.com/mozilla-b2g/gaia/commit/77b86130954fb5769c9b49f3184b37d1d9148225
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → FxOS-S7 (18Sep)
Keywords: checkin-needed
Depends on: 1208478
Depends on: 1213083
You need to log in before you can comment on or make changes to this bug.