994823 - [Messages] Only one notification per thread is displayed at one time

Status: RESOLVED DUPLICATE of bug 1082546

(Firefox OS Graveyard :: Gaia::SMS, defect)

Description

[Julien Wajsberg [:julienw]]

STR:
* receive 2 SMS from the same number, with the SMS app closed

Expected:
* 2 notifications are displayed

Actual:
* 1 notification only is displayed, from the last received message

What happens is that we tag the notification with the thread id, so the older notification gets replaced by the new one, instead of having another one.

While I think it could be a good thing, I think that in that case we'd want to change the notification text, saying that there are 2 unread messages for example.

NI Omega for 2 questions:
* should we restore the previous way of working as a first step
* should we implement a smarter way of doing multiple notifications as a second step

If we want to fix the regression (from bug 855165) then it will be probably 1.4+, but I'm waiting for Omega's answer before requesting the flag.

Comment 1

[Omega Feng [:Omega] [:馮於懋]]

OK, let's do this:

1 message:
[ic] %SenderName        %TimeStamp
[on] %Message

2 messages:
[ic] %SenderName        %TimeStamp
[on] 2 unread messages

Comment 2

[Julien Wajsberg [:julienw]]

Omega, I think we can't do this for 1.4 because of the string freeze, but we can definitely do this for 1.5, thanks :)

Do you want that we go back to the previous behavior for 1.4, or is the current behavior good enogh for you?

Comment 3

[Omega Feng [:Omega] [:馮於懋]]

IMO, for 1.4, the current behavior is good enough. No need to go back. Thanks! :)

Comment 4

[Julien Wajsberg [:julienw]]

oki, I hope partners won't come later to block on it ;)