972150 - [DSDS][Gaia][Sim Manager] support always ask feature

Status: RESOLVED FIXED

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

Description

[EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)]

In 1.4, we have to add one more feature which is `always ask` option on `outgoing call` and `outgoing message` select.

Comment 1

[EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)]

Attachment: patch

Comment 2

[EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)]

Comment on attachment 8375375 [details] [review]
patch

Hi Anthony,

Not sure whether this change is ok for you or not, so I may need your feedback for this patch.

Thanks :)

Comment 3

[EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)]

(In reply to EJ Chen [:eragonj][:小龍哥] from comment #2)
> Comment on attachment 8375375 [details] [review]
> patch
> 
> Hi Anthony,
> 
> Not sure whether this change is ok for you or not, so I may need your
> feedback for this patch.
> 
> Thanks :)

Please apply this patch to test your related works on dialer app ! Thanks :)

Comment 4

[Anthony Ricaud (:rik)]

Comment on attachment 8375375 [details] [review]
patch

I haven't written code that uses those settings yet so I can't really say it works for us. I think it should be ok.

needinfo-ing Doug who has some code locally.

Comment 5

[EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)]

(In reply to Anthony Ricaud (:rik) from comment #4)
> Comment on attachment 8375375 [details] [review]
> patch
> 
> I haven't written code that uses those settings yet so I can't really say it
> works for us. I think it should be ok.
> 
> needinfo-ing Doug who has some code locally.

Yeah ok ! Thanks for your quick response. Let's wait Doug's comments !

Comment 6

[Doug Sherk (:drs) (inactive)]

I just tried this, and it worked great for me! Thanks for this patch.

Comment 7

[EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)]

(In reply to Doug Sherk (:drs) from comment #6)
> I just tried this, and it worked great for me! Thanks for this patch.

Thanks Doug ! 

I would start the reviewing process for the patch !! :P

Comment 8

[EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)]

Comment on attachment 8375375 [details] [review]
patch

Hey Arthur,

this patch is finally ready for reviewing ! 

As you may see, Doug has tested this patch and works well for them. Besides, I also get green from Travis ! 

Please help me review this ~50 lines patch :)

Big thanks !!!!

Comment 9

[Arthur Chen [:arthurcc][inactive after 6/30]]

Comment on attachment 8375375 [details] [review]
patch

r=me. Thanks!

Comment 10

[EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)]

Thanks Arthur & all, 

this patch was merged on Gaia/master : 691651a19cad6ed6731f0a234f70fe5da2ff3bae