Closed
Bug 1092335
Opened 10 years ago
Closed 10 years ago
Ditch our upgrade nag bar in favor of Tabzilla's
Categories
(support.mozilla.org :: Code Quality, task, P3)
support.mozilla.org
Code Quality
Tracking
(Not tracked)
RESOLVED
FIXED
2014Q4
People
(Reporter: mythmon, Assigned: rrosario)
References
Details
(Whiteboard: u=dev c=general p=1 s=2014.20)
Tabzilla just landed support for version nagging[1]. We should use it instead of ours so that we can not worry about UA parsing and ESR det tion(which tabzilla will get soon) and what the latest Firefox is (at least for version nags).
I bet this will be pretty easy. The documentation for this [2] says we just have to add a data-attribute to the script tag we use for Tabzilla, and then we would need to remove all of our version notification code. Easy peasy.
[1] https://bugzilla.mozilla.org/show_bug.cgi?id=935719
[2] https://github.com/mozilla/bedrock/blob/master/docs/tabzilla.rst
Comment 1•10 years ago
|
||
This makes good sense!
Comment 2•10 years ago
|
||
Yes, let's do this!
Updated•10 years ago
|
Priority: -- → P3
Target Milestone: --- → 2014Q4
Assignee | ||
Comment 3•10 years ago
|
||
I love deleting code first thing on a Monday morning. Grabbing into current sprint.
Assignee: nobody → rrosario
Whiteboard: u=dev c=general p=1 s=2014.21 → u=dev c=general p=1 s=2014.20
Assignee | ||
Comment 4•10 years ago
|
||
In a pull request:
https://github.com/mozilla/kitsune/pull/2203
Assignee | ||
Comment 5•10 years ago
|
||
Assignee | ||
Comment 6•10 years ago
|
||
deployed to prod.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 7•10 years ago
|
||
(In reply to Ricky Rosario [:rrosario, :r1cky] from comment #6)
> deployed to prod.
Nice! "That was easy!"
You need to log in
before you can comment on or make changes to this bug.
Description
•