limit notifications at statup.

Bug #1079865 reported by Robert Bruce Park on 2012-11-16
This bug affects 6 people
Affects Status Importance Assigned to Milestone
Robert Bruce Park
friends (Ubuntu)

Bug Description

The notify code that we're about to land has a tendency to spam a ton of notifications at startup, which is probably not helpful when users are just turning their computer on -- they're probably more interested in the wireless notification or whatever other boot-time things are happening.

So we need to come up with a way to suppress the notifications at launch, however this gets a bit tricky; the naive approach of initializing a flag to false, and then setting it true at the end of the first refresh would work for the moment, but this would end up being just another bit of code that will break when we eventually move to the event-based architecture. So with not-long-running-process architecture in mind, we need to figure out a way to suppress notifications shortly after boot-time.

The only thing I can think of, although this idea probably needs some finessing, is to instead check the system uptime, and only display notifications if uptime is greater than, say, 30 minutes.

Any thoughts on a) how to reliably check the system uptime in python, and b) what the correct length of notification-suppression time should be?

Related branches

Changed in friends:
status: Triaged → Fix Committed
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:friends at revision 195, scheduled for release in friends, milestone Unknown

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package friends - 0.1.3daily13.04.17~13.04-0ubuntu1

friends (0.1.3daily13.04.17~13.04-0ubuntu1) raring; urgency=low

  [ Robert Bruce Park ]
  * No ConsumerKey in _get_oauth_headers (LP: #1169018)
  * doesn't resolve destination of links (LP: #1166080)
  * limit notifications at statup. (LP: #1079865)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 195

friends (0.1.3daily13.04.16.1~13.04-0ubuntu1) raring; urgency=low

  [ Ken VanDine ]
  * debian/control
    - make the service plugins depend on the appropriate account plugins

  [ Robert Bruce Park ]
  * Sender information empty for retweets (LP: #1168427)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 190
 -- Ubuntu daily release <email address hidden> Wed, 17 Apr 2013 05:03:06 +0000

Changed in friends (Ubuntu):
status: New → Fix Released
Changed in friends:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers