Closed Bug 1239914 Opened 9 years ago Closed 9 years ago

[TV][2.5] Fallback to old system banner

Categories

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

Product:
Firefox OS Graveyard
Component:
Gaia::TV::System
Platform:
ARM
Gonk (Firefox OS)
Type:
defect

Tracking

(blocking-b2g:2.5+, b2g-v2.5 verified, b2g-master verified)

Status:
VERIFIED FIXED
Project Flags:
blocking-b2g 2.5+
Tracking Flags:
Tracking Status
b2g-v2.5 --- verified
b2g-master --- verified

People

(Reporter: rickychien, Assigned: rickychien)

References

Details

(Whiteboard: [ft:conndevices][partner-cherry-pick])

Attachments

(3 files)

[gaia] mozilla-b2g:revert-33716-extendSystemBanner > mozilla-b2g:master
46 bytes, text/x-github-pull-request
jocheng
: approval-gaia-v2.5+
Details | Review
Smart-component PR
55 bytes, text/x-github-pull-request
Details | Review
Visual spec 2016-01-13
84 bytes, text/plain
Details
Latest visual spec (updated by 1/13) in bug 1235128 fallback to old style system banner so I will revert the patch of bug 1235128 and uplift to v2.5 as well.
I'm also revert patch for smart-components.
[Blocking Requested - why for this release]:
blocking-b2g: --- → 2.5?
Comment on attachment 8708194 [details] [review] [gaia] mozilla-b2g:revert-33716-extendSystemBanner > mozilla-b2g:master Luke, I'd like to inform you the latest visual spec changed and decided to fallback old style system banner.
Attachment #8708194 - Flags: review?(lchang)
Josh, visual spec decided to fallback old style system banner so bug 1235128 will be backed out here and uplift to v2.5 branch.
Blocks: TV_Marketplace_2.5
Flags: needinfo?(jocheng)
Priority: -- → P2
Blocks: 1239907
blocking-b2g: 2.5? → 2.5+
Flags: needinfo?(jocheng)
(In reply to Ricky Chien [:rickychien] from comment #6) > Josh, visual spec decided to fallback old style system banner so bug 1235128 > will be backed out here and uplift to v2.5 branch. Thanks for the heads up. Please raise uplift request when you got r+.
Attachment #8708194 - Flags: review?(lchang)
Reverted in gaia master https://github.com/mozilla-b2g/gaia/commit/6dfb1a87132ca48463ed50797a33e6027493330d Reverted in smart-components https://github.com/smart-components/smart-banner/commit/eaa9e8fd7a3e39556ab54f4bea3d853c8458d7c9
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment on attachment 8708194 [details] [review] [gaia] mozilla-b2g:revert-33716-extendSystemBanner > mozilla-b2g:master [Approval Request Comment] [Bug caused by] (feature/regressing bug #): revert bug 1235128 [User impact] if declined: fall back to old system banner UI [Testing completed]: verified manually on local [Risk to taking this patch] (and alternatives if risky): none [String changes made]: added-to-app deleted-from-apps preview-app
Attachment #8708194 - Flags: approval-gaia-v2.5?
Comment on attachment 8708194 [details] [review] [gaia] mozilla-b2g:revert-33716-extendSystemBanner > mozilla-b2g:master Approve for TV 2.5
Attachment #8708194 - Flags: approval-gaia-v2.5? → approval-gaia-v2.5+
Whiteboard: [ft:conndevices][partner-cherry-pick]
has conflicts uplifting to 2.5 - could you take a look and maybe provide a 2.5 PR request ? Thanks!
Flags: needinfo?(rchien)
Patch link generated by Github commit: https://github.com/rickychien/gaia/commit/34f4fc8183c0c289ee1db117711cada0d1256fb6.patch Hi, please take this patch, thanks!
Flags: needinfo?(rchien) → needinfo?(cbook)
Hi, this results on 2.5 in problems: adding 25.patch to series file applying 25.patch unable to find 'tv_apps/smart-system/bower_components/smart-banner/script.js' for patching 3 out of 3 hunks FAILED -- saving rejects to file tv_apps/smart-system/bower_components/smart-banner/script.js.rej unable to find 'tv_apps/smart-system/bower_components/smart-banner/style.css' for patching 3 out of 3 hunks FAILED -- saving rejects to file tv_apps/smart-system/bower_components/smart-banner/style.css.rej unable to find 'tv_apps/smart-system/index.html' for patching 4 out of 4 hunks FAILED -- saving rejects to file tv_apps/smart-system/index.html.rej unable to find 'tv_apps/smart-system/js/app_install_manager.js' for patching 3 out of 3 hunks FAILED -- saving rejects to file tv_apps/smart-system/js/app_install_manager.js.rej unable to find 'tv_apps/smart-system/js/browser_context_menu.js' for patching 8 out of 8 hunks FAILED -- saving rejects to file tv_apps/smart-system/js/browser_context_menu.js.rej unable to find 'tv_apps/smart-system/js/interactive_notifications.js' for patching 2 out of 2 hunks FAILED -- saving rejects to file tv_apps/smart-system/js/interactive_notifications.js.rej unable to find 'tv_apps/smart-system/js/preview_window.js' for patching 3 out of 3 hunks FAILED -- saving rejects to file tv_apps/smart-system/js/preview_window.js.rej unable to find 'tv_apps/smart-system/js/system_banner.js' for patching 2 out of 2 hunks FAILED -- saving rejects to file tv_apps/smart-system/js/system_banner.js.rej unable to find 'tv_apps/smart-system/locales/system.en-US.properties' for patching 2 out of 2 hunks FAILED -- saving rejects to file tv_apps/smart-system/locales/system.en-US.properties.rej unable to find 'tv_apps/smart-system/style/notifications/notifications.css' for patching 2 out of 2 hunks FAILED -- saving rejects to file tv_apps/smart-system/style/notifications/notifications.css.rej unable to find 'tv_apps/smart-system/style/system_banner/system_banner.css' for patching 1 out of 1 hunks FAILED -- saving rejects to file tv_apps/smart-system/style/system_banner/system_banner.css.rej could you take a look ? : )
Flags: needinfo?(cbook) → needinfo?(rchien)
Hi Tomcat, How do you apply the patch? Following steps works on my local: 1. checkout latest v2.5 branch (I can see latest commit is e36230a7b19ef5525543ae234408869b38ca65a9) 2. curl -L https://github.com/rickychien/gaia/commit/34f4fc8183c0c289ee1db117711cada0d1256fb6.patch | git am My console output: % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 100 26680 0 26680 0 0 10798 0 --:--:-- 0:00:02 --:--:-- 10801 Applying: Bug 1239914 - [TV][2.5] Fallback to old system banner
Flags: needinfo?(rchien) → needinfo?(cbook)
ah never did this that way :) applied and pushed that patch in https://github.com/mozilla-b2g/gaia/commit/53ba710af4baa6ea89f07f5d4bca36dc05476136
status-b2g-v2.5: affected → fixed
Flags: needinfo?(cbook)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: