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 660882 - Drop the notification from EnsureCredentials
Drop the notification from EnsureCredentials
Status: RESOLVED FIXED
Product: gnome-online-accounts
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: GNOME Online Accounts maintainer(s)
GNOME Online Accounts maintainer(s)
: 773386 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-10-04 14:11 UTC by David Zeuthen (not reading bugmail)
Modified: 2016-10-24 13:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
daemon: Do not use notifications when an account needs attention (13.94 KB, patch)
2013-03-05 12:21 UTC, Debarshi Ray
committed Details | Review

Description David Zeuthen (not reading bugmail) 2011-10-04 14:11:53 UTC
An application might want to display the error "Can't connect to GoogleTalk account" themselves instead of having the user bombarded with a generic

 "An online account needs attention"

for example

 http://people.freedesktop.org/~david/goa-expired-credentials.png

For example the design of the Documents application appear to call for this feature

 http://live.gnome.org/Design/Apps/Documents

and from IRC it appears empathy wants to use this as well.

It's worth noting that the app can already do this - the app can simply just avoid calling EnsureCredentials() but just call e.g. OAuth.GetAccessToken() and then try to use the token. If either OAuth.GetAccessToken() fails or the the web-service returns an "Not Authorized" error when using the token, the app knows the token is bad and the user needs to reauthenticate.
Comment 1 Debarshi Ray 2013-02-07 17:42:29 UTC
This is something that the designers have been asking for too. In fact, after talking to the designers, I think we don't need a quieter version of EnsureCredentials, but just make what we have now quieter.

I am CCing the Allan and Jon, and the maintainers of the affected applications, because it will need changes in the way the applications handle this failure.
Comment 2 Debarshi Ray 2013-02-07 17:46:08 UTC
I think I have covered everybody -- Empathy, Documents and Evolution.
Comment 3 Debarshi Ray 2013-02-07 17:47:33 UTC
I think gnome-shell's inbuilt Telepathy client is the worst offender. So we need to figure out how to show the failure in the shell as well. (Adding fmuellner)
Comment 4 Florian Müllner 2013-02-07 18:06:14 UTC
(In reply to comment #3)
> I think gnome-shell's inbuilt Telepathy client is the worst offender. So we
> need to figure out how to show the failure in the shell as well.

As it came up at fosdem, I've thought about it a bit. My suggestion would be to use an error icon for the user status, make the presence combo box insensitive, and provide a link to the online accounts panel in the user menu.
(Of course we'll need to figure out how we actually detect that case, given that we don't query goa ourselves but via telepathy ...)
Comment 5 Debarshi Ray 2013-03-05 12:12:16 UTC
Editing the subject to reflect the fact that we don't want a quieter version of EnsureCredentials, but want to drop the notification altogether.
Comment 6 Debarshi Ray 2013-03-05 12:21:01 UTC
Created attachment 238122 [details] [review]
daemon: Do not use notifications when an account needs attention
Comment 7 Debarshi Ray 2013-03-05 12:26:54 UTC
Now it is over to the applications to deal with it.
Comment 8 Debarshi Ray 2016-10-24 12:41:05 UTC
(In reply to Debarshi Ray from comment #1)
> This is something that the designers have been asking for too. In fact,
> after talking to the designers, I think we don't need a quieter version of
> EnsureCredentials, but just make what we have now quieter.

For the sake of leaving a paper trial, the problem with the desktop notification was that various background daemons kept triggering this error while the user is not consciously using the account. eg., evolution-data-server tries to use the calendar while the user is in the middle of a presentation. This is distracting. Asking the relevant applications to handle it avoids this problem, and there is a clear indication as to what is causing it.
Comment 9 Pablo Estigarribia 2016-10-24 13:07:29 UTC
*** Bug 773386 has been marked as a duplicate of this bug. ***