Closed
Bug 928292
Opened 11 years ago
Closed 11 years ago
[DSDS][Gaia] System should handle multiple sim card locks
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(blocking-b2g:1.3+, firefox28 fixed, b2g-v1.3 fixed)
People
(Reporter: arthurcc, Assigned: alive)
References
Details
(Whiteboard: [L2][Blocked by devices])
Attachments
(1 file)
- When users unlock the device, dialogs that allow users to unlock multiple sim cards should be displayed.
- When users launch apps that need a locked sim card, an unlocking dialog should also be displayed.
Hey Sandip, if both the SIMs in DSDS are PIN locked wouldn't we need support for this bug to unlock both SIMs? If yes, then this should be 1.3+
Flags: needinfo?(skamat)
Updated•11 years ago
|
blocking-b2g: 1.3? → 1.3+
Comment 2•11 years ago
|
||
Arthur, do we know when we could finish this bug? Thanks!
Updated•11 years ago
|
Flags: needinfo?(arthur.chen)
Reporter | ||
Updated•11 years ago
|
Flags: needinfo?(arthur.chen)
Target Milestone: --- → 1.3 Sprint 5 - 11/22
Updated•11 years ago
|
Blocks: b2g-dsds-l2
Comment 3•11 years ago
|
||
(In reply to Anshul from comment #1)
> Hey Sandip, if both the SIMs in DSDS are PIN locked wouldn't we need support
> for this bug to unlock both SIMs? If yes, then this should be 1.3+
Correct. It is for 1.3.
Flags: needinfo?(skamat)
Updated•11 years ago
|
Target Milestone: 1.3 Sprint 5 - 11/22 → 1.3 Sprint 6 - 12/6
Comment 4•11 years ago
|
||
Plan to land it in the 2nd week of sprint 6.
Whiteboard: [L2] → [L2][Blocked by devices]
Comment 5•11 years ago
|
||
The comment #4 is still valid.
Assignee | ||
Updated•11 years ago
|
Blocks: SIMPIN-dialog
Comment 7•11 years ago
|
||
What's the status now?
Assignee | ||
Comment 8•11 years ago
|
||
Have a working patch, writing tests now.
Assignee | ||
Comment 9•11 years ago
|
||
Rock DSDS!
Attachment #8343041 -
Flags: review?(timdream)
Attachment #8343041 -
Flags: feedback?(arthur.chen)
Reporter | ||
Updated•11 years ago
|
Attachment #8343041 -
Flags: feedback?(arthur.chen) → feedback+
Comment 10•11 years ago
|
||
Comment on attachment 8343041 [details]
DSDS SIM PIN
Thank you for taking the initiative and helping out!
Attachment #8343041 -
Flags: review?(timdream) → review+
Comment 12•11 years ago
|
||
We really need to know when the bug can be landed, because DSDS is a committed feature. Can you help to provide the ETA? Thank you.
Assignee | ||
Comment 13•11 years ago
|
||
Travis is green now, merging.
https://travis-ci.org/mozilla-b2g/gaia/builds/15206219
Assignee | ||
Comment 14•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
status-firefox28:
--- → affected
Assignee | ||
Comment 15•11 years ago
|
||
Request the approval of uplifting this patch to the v1.3 branch.
This blocks one of the major DSDS features and also partner's tests. We have
unit tests for the modified part (sim_pin_dialog) and also the tests for the newly
added module. This feature is also covered by moztraps as it is part of the
v1.3 user story.
Flags: needinfo?(fabrice)
Comment 16•11 years ago
|
||
Alive, I believe Bug 946603 will also block this bug, am I correct?
Flags: needinfo?(alive)
Assignee | ||
Comment 17•11 years ago
|
||
Yes for: the retryCount=-1 always is a regression.
Flags: needinfo?(alive)
Comment 18•11 years ago
|
||
(In reply to Alive Kuo [:alive][NEEDINFO] from comment #15)
> Request the approval of uplifting this patch to the v1.3 branch.
>
> This blocks one of the major DSDS features and also partner's tests. We have
> unit tests for the modified part (sim_pin_dialog) and also the tests for the
> newly
> added module. This feature is also covered by moztraps as it is part of the
> v1.3 user story.
Ok, feel free to merge.
Flags: needinfo?(fabrice)
Comment 19•11 years ago
|
||
What's the ETA on uplifting this to 1.3?
Assignee | ||
Comment 20•11 years ago
|
||
Need followup 928852 to be 1.3+ as well in order not to break 1.3
Assignee | ||
Comment 21•11 years ago
|
||
c0aa854489c102fad8ea7deac68f4e36fbe39e2a
v1.3
Comment 22•11 years ago
|
||
(In reply to Alive Kuo [:alive][NEEDINFO] from comment #20)
> Need followup 928852 to be 1.3+ as well in order not to break 1.3
I saw bug 928852 is duplicate to bug 928847, which is RESOLVED INVALID.
What's the plan we can take to get it landed in 1.3?
Flags: needinfo?(alive)
Assignee | ||
Comment 23•11 years ago
|
||
I put the wrong bug number :(
It's bug 948852 and already 1.3+
THX!
Flags: needinfo?(alive)
Updated•11 years ago
|
status-b2g-v1.3:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•