Closed
Bug 1061750
Opened 10 years ago
Closed 10 years ago
Single Variant - Build: Generate the configuration file for bookmarks
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(blocking-b2g:2.1+, b2g-v2.1 fixed, b2g-v2.2 fixed)
People
(Reporter: macajc, Assigned: macajc)
References
Details
(Whiteboard: [systemsfe])
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
yurenju
:
review+
albert
:
review+
fabrice
:
approval-gaia-v2.1+
|
Details |
Generated the configuration file needed by the browser single variant configuration implemented in bug 940647 in a similar way to the rest of the single variant customizations
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → carmen.jimenezcabezas
Assignee | ||
Comment 1•10 years ago
|
||
Comment 2•10 years ago
|
||
Wow. So were we doing /anything/ with the gaia/customization/browser.json file? That's what I was going off of for bug 940647.
Comment 3•10 years ago
|
||
[Blocking Requested - why for this release]: Without this, bookmark customization will never load the correct file, so we'll never actually customize bookmarks.
blocking-b2g: --- → 2.1?
Updated•10 years ago
|
Component: Gaia → Gaia::System::Browser Chrome
Updated•10 years ago
|
blocking-b2g: 2.1? → 2.1+
Updated•10 years ago
|
Whiteboard: [systemsfe]
Updated•10 years ago
|
Component: Gaia::System::Browser Chrome → Gaia
Updated•10 years ago
|
Target Milestone: --- → 2.1 S4 (12sep)
Assignee | ||
Updated•10 years ago
|
Attachment #8482853 -
Flags: review?(yurenju.mozilla)
Attachment #8482853 -
Flags: review?(alberto.crespellperez)
Comment 4•10 years ago
|
||
Comment on attachment 8482853 [details]
V1 Proposed patch
Nice, thank you Carmen!
I leave a comment in github, it is just a proposal..
As I saw tbpl not launched automatically I did it for you:
https://tbpl.mozilla.org/?tree=Try&rev=be0c90bb7b53
Attachment #8482853 -
Flags: review?(alberto.crespellperez) → review+
Comment 5•10 years ago
|
||
Comment on attachment 8482853 [details]
V1 Proposed patch
code looks good but we got a build error on try server but it looks an intermittent error, I have re-run it.
another thing is seem we lack some document for variant.json on MDN[1], could you also update the document?
and r=yurenju, thank you!
[1] https://developer.mozilla.org/en-US/Firefox_OS/Developing_Firefox_OS/Market_customizations_guide
Attachment #8482853 -
Flags: review?(yurenju.mozilla) → review+
Comment 6•10 years ago
|
||
Assignee | ||
Comment 7•10 years ago
|
||
I've retriggered the Gb job several times and one of them at least is green.
I don't think the errors are related with this patch at all (in fact, I get errors when triggering a try run without my patch also).
I haven't merged it yet just in case you don't want me to, but I think it should be safe to merge (at least it won't break the tests more :)). Can I merge this (or you if you wish) or do you want me to check something else before?
Flags: needinfo?(yurenju.mozilla)
Comment 8•10 years ago
|
||
please land it, the root cause of red Gb has been fixed on bug 1061563, so it shoud be gone if rebase the pr.
Flags: needinfo?(yurenju.mozilla)
Assignee | ||
Comment 9•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•10 years ago
|
||
Comment on attachment 8482853 [details]
V1 Proposed patch
[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): Bug 1061804 - Browser migration operator variant followup
[User impact] if declined: Without this, bookmark customization will never load the correct file, so we'll never actually customize bookmarks
[Testing completed]: Unit tests in gaia
[Risk to taking this patch] (and alternatives if risky): Low
[String changes made]: None
Attachment #8482853 -
Flags: approval-gaia-v2.1?
Updated•10 years ago
|
Attachment #8482853 -
Flags: approval-gaia-v2.1? → approval-gaia-v2.1+
Comment 11•10 years ago
|
||
status-b2g-v2.1:
--- → fixed
status-b2g-v2.2:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•