800271 - B2G STKUI: Add support to STK_CMD_SET_UP_IDLE_MODE_TEXT

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

Add support to STK_CMD_SET_UP_IDLE_MODE_TEXT on STK app.

Comment 1

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

As Yoshi informed me:

SET_UP_IDLE_MODE_TEXT is to show the text in idle mode (i.e. screen off),
whereas REFRESH is to tell ME to remove the idle mode text()

Comment 2

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

I'd been talking with Rafael and we decided to show this messages with the attention screen (notifications) and ignore REFRESH command to remove it since it'll be removed automatically when the users attends the message.

Comment 4

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

(In reply to Fernando R. Sela from comment #2)
> I'd been talking with Rafael and we decided to show this messages with the
> attention screen (notifications) and ignore REFRESH command to remove it
> since it'll be removed automatically when the users attends the message.

To clarify, REFRESH won't be ignored, I talked with Beatriz and we shall follow the specs so we should remove the notification message.

Comment 5

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

Attachment: Patch to support IDLE messages commands

This is a preview since this patch is asking the user to ALLOW the notification and shall be transparent to him.

Also it's needed to add a remove method but I'm not sure if remove is supported by mozNotification ...

Comment 6

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

Comment on attachment 671809 [details] [diff] [review]
Patch to support IDLE messages commands

The issue with permissions had been solved in my handset, I think this patch is prepared to land.

The message remove is pending (TODO)

----

NOTE: If blocking-basecamp+ is set, just land it for now.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: 
Testing completed: 
Risk to taking this patch (and alternatives if risky):

Comment 7

[Mike Connor [:mconnor]]

Per b2g-drivers, no feature work will be marked as blocking, and require driver approval to land.  Marking as blocking-, please get approval from Vivien before landing.

Comment 8

[Doug Turner (:dougt)]

Fernando, Please explain why this would depend on 782211.  Removing for now.

Comment 9

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

(In reply to Doug Turner (:dougt) from comment #8)
> Fernando, Please explain why this would depend on 782211.  Removing for now.

We need the improvements on 782211 in order to be able to remove notifications when the REFRESH command is sent by the UICC.

But after the discussion in the webapi list this won't be landed before V1 so I agree to remove this dependency.

Comment 10

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

Comment on attachment 671809 [details] [diff] [review]
Patch to support IDLE messages commands

Hi Vivien,

It's pending the remove notification on REFRESH command but I think we cann't support it now since mozNotification API don't support it ... probably in next release we can support this one.

Now is commented as TO-DO.  In order to avoid "noise" changing the patch for only this comment I ask you for approval to land it and I'll rebase it and also change this comment to a more clear one before the final land :p

--


NOTE: If blocking-basecamp+ is set, just land it for now.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: 
Testing completed: 
Risk to taking this patch (and alternatives if risky):

Comment 11

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 671809 [details] [diff] [review]
Patch to support IDLE messages commands

Review of attachment 671809 [details] [diff] [review]:
-----------------------------------------------------------------

Part of of a late feature work. This code only alters the behavior of the icc part of settings with small changes. a=me.

Comment 12

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Doug Turner (:dougt) from comment #8)
> Fernando, Please explain why this would depend on 782211.  Removing for now.

I believe this is because they want to use notification.close.

Comment 13

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

Attachment: Link to GitHub PR

Patch approved. PR link

Comment 14

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

Attachment: Link to GitHub PR

Comment 15

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

Merged: https://github.com/mozilla-b2g/gaia/pull/5928
https://github.com/mozilla-b2g/gaia/commit/b98fb062612ee58fefdeebc27c0d42c3751dcaf2

Comment 16

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

Attachment: Syntax error fix

A little syntax fix

--

NOTE: If blocking-basecamp+ is set, just land it for now.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: 
Testing completed: 
Risk to taking this patch (and alternatives if risky):

Comment 17

[Natalya Kot [:nkot]]

fix verified in Build 20130113070202 - the text in shown when in idle mode