Closed Bug 928295 Opened 8 years ago Closed 8 years ago

[DSDS][Gaia] Settings app should support sim security settings of multiple sim cards

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:1.3+, b2g-v1.3 fixed)

RESOLVED FIXED
1.3 Sprint 6 - 12/6
blocking-b2g 1.3+
Tracking Status
b2g-v1.3 --- fixed

People

(Reporter: arthurcc, Assigned: eragonj)

References

Details

Attachments

(1 file)

Users should be able to set sim security for multiple sim cards.
blocking-b2g: 1.3? → 1.3+
Arthur, is it something you can help? Thanks!
Flags: needinfo?(arthur.chen)
Sandip, can we move it to 1.4?
Flags: needinfo?(skamat)
Arthur, are you going to take this bug?
Assignee: nobody → arthur.chen
Flags: needinfo?(arthur.chen)
Target Milestone: --- → 1.3 Sprint 6 - 12/6
(In reply to Kevin Hu [:khu] from comment #2)
> Sandip, can we move it to 1.4?

This story looks to be a duplicate of bug 926350, which is marked for 1.3.
Flags: needinfo?(skamat)
You're right, Sandip. 
I think this bug should block bug 926350.
Blocks: 926350
No longer blocks: 918533
Duplicate of this bug: 935830
Assignee: arthur.chen → ejchen
I will integrate existing sim security panel into SimCardManager
Attached file WIP
I just moved Sim Card Security panel into Sim Manager. Right now, it works on single sim structure and views get rendered in DSDS structure. But still have to use new DSDS API to make it work for DSDS.
Plan to land it in the 1st week of sprint 6.
Hi Anshul, 

we have tested a lot on this part but it seems that there are some cardState issue now.

Can you help us to give it a try there ? thanks !
Flags: needinfo?(anshulj)
Let's move forward without having comments from Anshul.
Flags: needinfo?(anshulj)
Forgot to say: my comment is the conclusion from the meeting with Anshul.
May I know if this is planed to land before the end of this week? Or, next week? Thanks.
Flags: needinfo?(ejchen)
Hi Kevin, 

I have WIP now but we may need more tests on stable devices to make sure it really works.

I think this bug would be postponed to next week. 

thanks.
Flags: needinfo?(ejchen)
Comment on attachment 829989 [details] [review]
WIP

Hi Arthur, 

I think this patch is ready to review. I am working on adding more tests for it and will ask Aknow for some details when airplane mode is on / off.

Thanks :)
Attachment #829989 - Flags: review?(arthur.chen)
Depends on: 944263
Hsin-Yi, when airplane mode is on (Gaia will call mobileConnection.setRadioEnabled), is it possible for users to set defaultId on outgoing data / outgoing call and outgoing messages ? 

Currently in UX spec, there is no design for this special case. Before asking UX about the flow, I want to know the technical details behind setting them to make sure users can set defaultId or not when airplane mode is on. 

Thanks :)
Flags: needinfo?(htsai)
(In reply to EJ Chen [:eragonj] from comment #17)
> Hsin-Yi, when airplane mode is on (Gaia will call
> mobileConnection.setRadioEnabled), is it possible for users to set defaultId
> on outgoing data / outgoing call and outgoing messages ? 
User can only dial up emergency call when airplane mode is on.
Per comment 18, it seems that we should still allow users to select a primary sim when airplane mode is enabled.
After asking HsinYi, it is possible to set defaultId when we are in airplane mode.
(In reply to EJ Chen [:eragonj] from comment #20)
> After asking HsinYi, it is possible to set defaultId when we are in airplane
> mode.

Agree with comment 20.
Flags: needinfo?(htsai)
Comment on attachment 829989 [details] [review]
WIP

Great work EJ! The patch works well. There are a few minor issues I commented in github need to be addressed before merging. Please request a review after they are fixed. Thanks!
Attachment #829989 - Flags: review?(arthur.chen)
Comment on attachment 829989 [details] [review]
WIP

Hi Arthur,

I updated all code based on your comments and they all work well on single sim & dsds. 

please help me review the code again, thanks :)
Attachment #829989 - Flags: review?(arthur.chen)
Comment on attachment 829989 [details] [review]
WIP

Great work! Please merge the code with travis green, thanks!
Attachment #829989 - Flags: review?(arthur.chen) → review+
Hi Fabrice, 

we are going to land this feature onto gaia master. please take a look when you have time. 

By the way, because right now gaia tree is all red, I will merge it later after weekend. 

Thanks :)
Flags: needinfo?(fabrice)
(In reply to EJ Chen [:eragonj] from comment #25)
> Hi Fabrice, 
> 
> we are going to land this feature onto gaia master. please take a look when
> you have time. 
> 
> By the way, because right now gaia tree is all red, I will merge it later
> after weekend. 
> 
> Thanks :)

Feel free to merge, thanks.
Flags: needinfo?(fabrice)
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.
Sorry for the late due to unstable testing environment on Travis and it made Gaia tree closed for few days. 

I just finally got green from Travis after rerun my tests for 1 more hour ... 

This patch is landed on :

gaia/master : 8a2b8d0e930aa16ec6d2271b336cfc0d531d05fa
gaia/v1.3 : c01f0ee5139194ffc3bced76fdf19c515f35304f

thanks all.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
(In reply to EJ Chen [:eragonj] from comment #28)
> Sorry for the late due to unstable testing environment on Travis and it made
> Gaia tree closed for few days. 
> 
> I just finally got green from Travis after rerun my tests for 1 more hour
> ... 
> 
> This patch is landed on :
> 
> gaia/master : 8a2b8d0e930aa16ec6d2271b336cfc0d531d05fa
> gaia/v1.3 : c01f0ee5139194ffc3bced76fdf19c515f35304f
> 
> thanks all.

Indeed great news, thanks EJ :)
Blocks: 953150
You need to log in before you can comment on or make changes to this bug.