809974 - Populate the Device Information->Last Updated field with the last update date/time

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Settings, defect, P1)

Description

[Lukas Blakk [:lsblakk] use ?needinfo]

I say date/time in here because we might have more than one update in a day so seeing 11-05/12:52 vs 11-05/2:43 would help with knowing what a stable channel user is on.

We'll need this more once we have FOTA enabled and users have to go from update to update in a linear fashion. It will tell us a) how far away from 'done' the update path is and b) allow the testers an at-a-glance way to talk about what update they are on that is not the build id (most people in the test community will not know what to do with a build id).

Comment 1

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Gaia triage : LAst Updated field is there.  We would need to fix or remove.  blocking +, P1.

Comment 3

[Marshall Culpepper [:marshall_law]]

Attachment: last update installed - v1

This sets a pref and setting, which Gaia uses to show in "Last Updated" under
the Settings app in "Device Info".

Comment 4

[[:fabrice] Fabrice Desré]

Comment on attachment 684093 [details] [diff] [review]
last update installed - v1

Review of attachment 684093 [details] [diff] [review]:
-----------------------------------------------------------------

Why do we need both the pref and the setting?

::: b2g/components/UpdatePrompt.js
@@ +152,5 @@
>    },
>  
>    showUpdateHistory: function UP_showUpdateHistory(aParent) { },
> +  showUpdateInstalled: function UP_showUpdateInstalled() {
> +    let installed = new Date().getTime();

Nit: use Date.now();

Comment 5

[Marshall Culpepper [:marshall_law]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/567bfba61e51

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/567bfba61e51

Comment 7

[Marshall Culpepper [:marshall_law]]

Attachment: Gaia pull request 6560

Comment 8

[Marshall Culpepper [:marshall_law]]

Reopening for gaia pull request as well

Comment 9

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 686615 [details] [review]
Gaia pull request 6560

Per Github comments re-assigning Evelyn for review.

Comment 10

[Evelyn Hung]

Comment on attachment 686615 [details] [review]
Gaia pull request 6560

r=me, r+ with comment addressed.

Comment 11

[Fabien Cazenave [:kaze]]

https://github.com/mozilla-b2g/gaia/commit/46baef6a4277dd52fd9e37b018bbc2af7e6e3de5

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/fcfcf8f40472
https://hg.mozilla.org/releases/mozilla-b2g18/rev/66f5143457d0