After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 725165 - Show the date/time when we last checked for updates
Show the date/time when we last checked for updates
Status: RESOLVED FIXED
Product: gnome-software
Classification: Applications
Component: General
3.11.x
Other Linux
: Normal normal
: ---
Assigned To: GNOME Software maintainer(s)
GNOME Software maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2014-02-25 17:37 UTC by Allan Day
Modified: 2014-03-11 08:02 UTC
See Also:
GNOME target: ---
GNOME version: 3.11/3.12


Attachments
updates: Show the time we last checked for updates (6.95 KB, patch)
2014-03-03 13:26 UTC, Kalev Lember
none Details | Review
updates: Show the time we last checked for updates (6.95 KB, patch)
2014-03-03 13:28 UTC, Kalev Lember
none Details | Review
Update the saved timestamp when manually refreshing updates (1.20 KB, patch)
2014-03-03 13:53 UTC, Kalev Lember
committed Details | Review
updates: Show the time we last checked for updates (10.45 KB, patch)
2014-03-10 17:55 UTC, Kalev Lember
committed Details | Review

Description Allan Day 2014-02-25 17:37:34 UTC
This will make the system more transparent. It will also be helpful, by informing people about how often we are checking for updates, and whether it would be interesting for them to perform a manual check.

The simplest thing to do would be to show a string in the updates panel, at the bottom.

If the check was done today, show the time, otherwise the date. eg. "Last checked: 13:40"
Comment 1 Richard Hughes 2014-02-25 17:40:12 UTC
You want this for 3.12?
Comment 2 Allan Day 2014-02-25 17:43:34 UTC
(In reply to comment #1)
> You want this for 3.12?

It would be nice but it seems a bit late in the cycle to be suggesting UI changes. This is an enhancement; it can wait.
Comment 3 Richard Hughes 2014-02-25 17:57:15 UTC
It's doable, it's just an extra string. Can you show on the mockup where the string should sit on the "has no updates" and the "has updates" and the "currently refreshing" page. Thanks.
Comment 4 Allan Day 2014-02-25 18:06:46 UTC
(In reply to comment #3)
> It's doable, it's just an extra string. Can you show on the mockup where the
> string should sit on the "has no updates" and the "has updates" and the
> "currently refreshing" page. Thanks.

Just show it for the no updates case. It should be towards the bottom of the page and be horizontally centered.
Comment 5 Kalev Lember 2014-03-03 13:26:15 UTC
Created attachment 270794 [details] [review]
updates: Show the time we last checked for updates

This adds a label with 'Last checked: HH:MM' towards the bottom of the
"has no updates" page.
Comment 6 Kalev Lember 2014-03-03 13:28:34 UTC
Created attachment 270796 [details] [review]
updates: Show the time we last checked for updates

This adds a label with 'Last checked: HH:MM' towards the bottom of the
"has no updates" page.
Comment 7 Kalev Lember 2014-03-03 13:33:45 UTC
Screenshot: https://people.gnome.org/~klember/gnome-software-updates-last-checked.png

I'm not sure how to best position the new text, and what size the font should be. Currently it's the same size as the "Software is up to date" text and positioned 32 px from the bottom. Help appreciated!
Comment 8 Kalev Lember 2014-03-03 13:53:07 UTC
Created attachment 270801 [details] [review]
Update the saved timestamp when manually refreshing updates

This ensures the "Last checked: HH:MM" label properly updates on
clicking the update refresh button.
Comment 9 Kalev Lember 2014-03-10 17:54:39 UTC
14:03 < kalev> aday, hughsie: in other news, do you guys want https://bugzilla.gnome.org/show_bug.cgi?id=725165 for 3.12? I've attached patches there.
14:03 < Services> Bug 725165: normal, Normal, ---, gnome-software-maint, NEW, Show the date/time when we last checked for updates
14:03 < hughsie> kalev, ahh yes, i completely forgot about that
14:05 < hughsie> aday, if https://people.gnome.org/~klember/gnome-software-updates-last-checked.png is okay to you
14:06 < aday> hughsie, the position looks good. i'd use the standard text size and weight
14:06 < hughsie> kalev, ^^
14:06 < aday> hughsie, kalev, also, it would be good to strip the leading 0 there
14:07 < hughsie> kalev, do we need a strig freeze exception for that?
14:07 < kalev> hughsie: we do, yep
14:08 < kalev> aday: cool, let me update it and get you a new screenshot
14:08 < hughsie> kalev, in which case, can you request it now please and with the acks and few changes please push
14:08 < kalev> sure
14:08  * hughsie is fighting with fonts :)
14:09 < kalev> aday: would it be better if it had the word "today" in there, as in "Last checked: today, 9:23 AM" ?
14:10 < aday> kalev, personally i don't feel that's necessary... what does it show for yesterday or other days?
14:13 < kalev> aday: for yesterday, it's "Last checked: day_as_number month_as_word year_including_century"
14:14 < kalev> pff, I guess it should be locale dependant
14:14 < aday> kalev, it gives the month and year? it would be nice if it could just say "Yesterday, 9:23 AM"
14:15 < kalev> sure, I can do that
14:18 < aday> kalev, and then if it's from before yesterday, you could do "two days ago", "one week ago", etc
14:19 < aday> kalev, i seem to recall us talking about having this is glib (?) at some point... it's a common thing to want to do
14:20 < kalev> maybe in libgnome-desktop
14:20 < kalev> but yeah, it should be in a common place, instead of getting duplicated all over different apps
Comment 10 Kalev Lember 2014-03-10 17:55:33 UTC
Created attachment 271456 [details] [review]
updates: Show the time we last checked for updates

This adds a label with 'Last checked: HH:MM' towards the bottom of the
"has no updates" page.
Comment 11 Kalev Lember 2014-03-10 18:02:27 UTC
The updated patch now tries to make the string more human readable, as per the review comments, and also uses the regular font. I am not entirely happy having all the time format strings here, but meh ... I guess we'll have to live with this for 3.12. This now also properly handles the format strings depending on whether the user has selected "AM / PM" or "24-hour" in system settings Date & Time panel.

https://people.gnome.org/~klember/gnome-software-updates-last-checked2.png
Comment 12 Kalev Lember 2014-03-11 08:01:57 UTC
Attachment 270801 [details] pushed as 38aa716 - Update the saved timestamp when manually refreshing updates
Attachment 271456 [details] pushed as 4f4d04d - updates: Show the time we last checked for updates

I've pushed these leaving out some of the new strings, at the request of the i18n team:

https://mail.gnome.org/archives/gnome-i18n/2014-March/msg00042.html