Page MenuHomePhabricator

Change "seenTime" format to ISO 8601 in Notifications front-end
Closed, ResolvedPublic

Description

Followup on T139993: Add 'seenTime' values for alert/message in the API when fetching list of notifications - we are deprecating MW timestamp, and starting to use ISO 8601 exclusively. The front-end should adjust accordingly.

Event Timeline

Mooeypoo created this task.Jul 22 2016, 3:59 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 22 2016, 3:59 PM

This was fixed by

All timestamps (seen time and notification item timestamps, etc) are now ISO8601 in UTC. The only place that localizes those timestamps are the item widgets.
The special page also uses localized times when dividing the notifications into days.

QA should probably concentrate on making sure all visible timestamps are properly localized in the popup and the special page, and that 'seenTime' works properly as well.
Local seen time should work now, but remote seenTime (in cross-wiki and the sidebar in the special page) will not yet work until all wikis are updated (probably next week)

Checked the displayed date format for correctness/completeness in betalabs 1.28.0-alpha (4737ea0).

jmatazzoni closed this task as Resolved.Aug 8 2016, 6:54 PM