1244020 - [TV][2.5] Reset and no longer count "open times" after app was added

Status: VERIFIED FIXED

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

Description

[Ricky Chien [:rickychien] (inactive)]

Attachment: marketplace-tv-160129.pdf

According UX spec P.29 as attachment, we should reset and no longer count "open times" after app was added because it will run into a weird scenario when user opens up website's preview window in X times and add and then remove that website by option key. Once user removes that website will close preview window instantly and prompt the "Add {app name} to Apps?" dialog after then. It's an unintuitive use case should be fixed.

Comment 1

[GitHub Autolander Bot]

Attachment: [gaia] rickychien:reset-open-times > mozilla-b2g:master

Comment 2

[Luke Chang [:lchang] (inactive)]

Comment on attachment 8714834 [details] [review]
[gaia] rickychien:reset-open-times > mozilla-b2g:master

Two comments:

1. Could you write a function for resetting the counter in PreviewWindow?
2. Make the timing more consistent. (e.q. right before showing "added to apps")

Comment 3

[Luke Chang [:lchang] (inactive)]

Comment on attachment 8714834 [details] [review]
[gaia] rickychien:reset-open-times > mozilla-b2g:master

Good to go!

Comment 4

[Ricky Chien [:rickychien] (inactive)]

Landed in master:

https://github.com/mozilla-b2g/gaia/commit/14ab87ef3ac912a949da1d214eb875c5798246a5

Comment 5

[Ricky Chien [:rickychien] (inactive)]

Comment on attachment 8714834 [details] [review]
[gaia] rickychien:reset-open-times > mozilla-b2g:master

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): new feature
[User impact] if declined: minor
[Testing completed]: verified manually on simulator
[Risk to taking this patch] (and alternatives if risky): none
[String changes made]: none

Comment 6

[Ricky Chien [:rickychien] (inactive)]

Attachment: patch uplifts to v2.5

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): new feature
[User impact] if declined: minor
[Testing completed]: verified manually on simulator
[Risk to taking this patch] (and alternatives if risky): none
[String changes made]: none

Comment 7

[Ricky Chien [:rickychien] (inactive)]

Please apply to v2.5 by this patch:

curl -L https://github.com/rickychien/gaia/commit/dd27921c5a851bf5805da3a87e59de9aa88c0d94.patch | git am

Comment 8

[Josh Cheng [:josh]]

Comment on attachment 8715637 [details]
patch uplifts to v2.5

Approve for TV 2.5

Comment 9

[Ricky Chien [:rickychien] (inactive)]

For uplifting to v2.5:

curl -L https://github.com/rickychien/gaia/commit/dd27921c5a851bf5805da3a87e59de9aa88c0d94.patch | git am

Comment 10

[Carsten Book [:Tomcat]]

Hi Ricky, i did this and got :

Applying: Bug 1244020 - [TV][2.5] Reset preview window open times
error: patch failed: tv_apps/smart-system/js/app_install_manager.js:45
error: tv_apps/smart-system/js/app_install_manager.js: patch does not apply
Patch failed at 0001 Bug 1244020 - [TV][2.5] Reset preview window open times
The copy of the patch that failed is found in: .git/rebase-apply/patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

Comment 11

[Ricky Chien [:rickychien] (inactive)]

curl -L https://github.com/rickychien/gaia/commit/6f1847582a080856c4c9cc669618a6b6a839e46c.patch | git am

Comment 12

[Carsten Book [:Tomcat]]

https://github.com/mozilla-b2g/gaia/commit/28880004bc5cd6e129d0a23b701a389753788ed0

Comment 13

[[e-mail]]

We verified on simulator version 2.6.20160221014426.
gaia:
https://github.com/mozilla-b2g/gaia/commit/a6ecae635719115aa72465efe522fdced3dd1d70
gecko:
https://hg.mozilla.org/mozilla-central/rev/af6356a3e8c56036b74ba097395356d9c6e6c5a3

If the user presses “Add”, the content will be add to “Web Apps” and the notification "Press option to ADD to APPS" will no longer appear.