1202360 - Bring Firefox Account handling modules back to TV system

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::TV::System, defect, P1)

Description

[yifan [:yifan][:yliao]]

Firefox Account modules were not included in the TV system app since the beginning of the TV project. Now we need Firefox Account sync features so we have to merge them back to TV system app.

Comment 1

[GitHub Autolander Bot]

Attachment: [gaia] begeeben:1202360_bring_firefox_account > mozilla-b2g:master

Comment 2

[yifan [:yifan][:yliao]]

Comment on attachment 8659105 [details] [review]
[gaia] begeeben:1202360_bring_firefox_account > mozilla-b2g:master

The patch brings the Firefox Account sign in feature in the system app to Firefox OS TV system. Please help to review it, thanks Rex!

Hi Fernando, I've added 1 line in shared/js/fxa_iac_client.js to enabled IAC connection with the TV smart system app. Please help to review it, thanks!

Comment 3

[Fernando Jiménez Moreno [:ferjm]]

Comment on attachment 8659105 [details] [review]
[gaia] begeeben:1202360_bring_firefox_account > mozilla-b2g:master

Thank you YiFan. LGTM.

This patch adds the FxA module exactly as it is on the phone. So I guess you'll need to adapt the UI so it looks good on the TV. Is there any bug filed for that?

Comment 4

[yifan [:yifan][:yliao]]

Thank you Fernando! It would be bug 1203898. We'll start working on it once visual design is completed.

Comment 5

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Comment on attachment 8659105 [details] [review]
[gaia] begeeben:1202360_bring_firefox_account > mozilla-b2g:master

First thank you for the effort of arranging these libraries into smart-system!
It looks good overall, but there are just 3 test cases failed; Let's fix them before landing. See some of my comments on Github. Thanks!

Comment 6

[yifan [:yifan][:yliao]]

Comment on attachment 8659105 [details] [review]
[gaia] begeeben:1202360_bring_firefox_account > mozilla-b2g:master

Thanks Rex! Fixed nits. Please review it, thanks!

Hi Ricky, the build test for GAIA_DEVICE_TYPE=tv in apps/system is removed, please help to review, thanks!

Comment 7

[Ricky Chien [:rickychien] (inactive)]

Comment on attachment 8659105 [details] [review]
[gaia] begeeben:1202360_bring_firefox_account > mozilla-b2g:master

r+ for build part changesets. 

A small question, I wonder why you need to put dev_apps/uitest in app.list?

Comment 8

[yifan [:yifan][:yliao]]

Thanks! The tests for fxa modules need uitest app to work.

Comment 9

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Comment on attachment 8659105 [details] [review]
[gaia] begeeben:1202360_bring_firefox_account > mozilla-b2g:master

Looks good to me.
Thanks a lot!

Comment 10

[yifan [:yifan][:yliao]]

Thanks! Merged into master:

https://github.com/mozilla-b2g/gaia/pull/31782