Code review comment for lp:~jonas-drange/account-polld/lp1570415

Revision history for this message
Bill Filler (bfiller) wrote :

tested, works for case if # rec messages < batch limit. But think there is a related condition that is not covered. If we recv > batchlimit # of messages, the notification shows "20 new messages" but seems we get two sets of vibrations. Guessing first one is for the message 0 and the next one is for the batch notification? In this case we should only vibrate once on the batch notification

review: Needs Fixing

« Back to merge proposal