howie [:howie] Reporter
	Description • 10 years ago

To implement a CMAS on/off toggle in Settings.

	howie [:howie] Reporter
	Comment 1 • 10 years ago

Hi Arthur, please reassign if needed, thanks.

	gasolin@mozilla.com Assignee
	Comment 2 • 10 years ago

Hi jenny, could you help provide related UI spec?

	Jenny Lee
	Comment 3 • 10 years ago

You can find it here: bug 1026685 attachment 8452848 [details] 
Note that all future spec update will be posted in the bug too, thanks!

	gasolin@mozilla.com Assignee
	Comment 4 • 10 years ago

Thanks jenny!

According to the spec, the requirement is add `Emergency Alert` toggle under `Message Settings` panel > `Cell Broadcast Messages` section.

	gasolin@mozilla.com Assignee
	Comment 5 • 10 years ago

@Bruce I'd like confirm the dependency between `Cell Broadcast` toggle and `Emergency Alert` toggle.

Does the following rule correct?

1. When Cell Broadcast is disabled, Emergency Alert is disabled as well
2. When Cell Broadcast is enabled, Emergency Alert get's its pref state and can be toggled

	gasolin@mozilla.com Assignee
	Comment 6 • 10 years ago

got the confirm in bug 1026685 comment 11

1. Two toggles, CB and CMAS
2. When CB on, user can turn on/off the CMAS
3. When CB off, CMAS is also off for sure.

I'd use 'ril.cellbroadcast.cmas.disabled' as mozSettings key in UI if no objection.

	gasolin@mozilla.com Assignee
	Comment 7 • 10 years ago

Implement above requirement, add following rule:

* When CB on, CMAS state will be retained to the state before CB off.
(if CMAS is on before CB off, after CB on, CMAS will be on as well)

with marionette test

	gasolin@mozilla.com Assignee
	Comment 8 • 10 years ago

Note this patch does not include `SIM settings` panel and the multiple SIMs sub panels described in UI spec

	gasolin@mozilla.com Assignee
	Comment 9 • 10 years ago

BTW, The wording is corrected by following Title wording rule described in bug 936398

	Arthur Chen [:arthurcc][inactive after 6/30]
	Comment 10 • 10 years ago

Comment on attachment 8464400 [details]
pull request redirect to github

Thanks for the patch. Please check my comments in github.

	gasolin@mozilla.com Assignee
	Comment 11 • 10 years ago

as bug 1026685 comment 21, The CMAS settings key would be 'cmas.enabled'

	gasolin@mozilla.com Assignee
	Comment 12 • 10 years ago

Comment on attachment 8464400 [details]
pull request redirect to github

The update includes

- Add CMAS disabled when CB off rule.
- Use asyncStorage to remember last CMAS state when CB off
- add marionette test for CB/CMAS dependency

	EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)
	Comment 13 • 10 years ago

Comment on attachment 8464400 [details]
pull request redirect to github

Hi Fred,

please check my comments on Github ! Thanks :)

	Arthur Chen [:arthurcc][inactive after 6/30]
	Comment 14 • 10 years ago

Comment on attachment 8464400 [details]
pull request redirect to github

Please check my comments in github, thanks.

	gasolin@mozilla.com Assignee
	Comment 15 • 10 years ago

Comment on attachment 8464400 [details]
pull request redirect to github

The logic is revised to make CMAS totally count on CBS, which lead to simpler code \o/

please kindly review it again

	Arthur Chen [:arthurcc][inactive after 6/30]
	Comment 16 • 10 years ago

Comment on attachment 8464400 [details]
pull request redirect to github

There is one last issue to be addressed before merging. AsyncStorage supports storing arrays, please check my github comments.

	gasolin@mozilla.com Assignee
	Comment 17 • 10 years ago

Comment on attachment 8464400 [details]
pull request redirect to github

changed to write array directly into asyncStorage, please kindly review it again

	Arthur Chen [:arthurcc][inactive after 6/30]
	Comment 18 • 10 years ago

Comment on attachment 8464400 [details]
pull request redirect to github

r=me with the nits addressed. Please squash the commits before merging, thanks!

	gasolin@mozilla.com Assignee
	Comment 19 • 10 years ago

TBPL test passed,
merged to gaia master https://github.com/mozilla-b2g/gaia/commit/aab6b29f304d4bea7fd6e03dc661f86f9a8d4e9b

thanks!