Closed
Bug 990976
Opened 11 years ago
Closed 11 years ago
[IMPORT] Move rest of dependant files to shared
Categories
(Firefox OS Graveyard :: Gaia::First Time Experience, defect)
Tracking
(b2g-v2.0 fixed)
RESOLVED
FIXED
1.4 S5 (11apr)
Tracking | Status | |
---|---|---|
b2g-v2.0 | --- | fixed |
People
(Reporter: fcampo, Assigned: fcampo)
References
Details
Attachments
(1 file)
To tackle all the rest of the import bugs we need first to move the files to shared, and later we'll do the same with functionality
Assignee | ||
Comment 1•11 years ago
|
||
Morphing the bug title to include the helping libraries, styles, css and locales needed for the import, but unrelated to specific import process.
Summary: [IMPORT] Move all dependant js files to shared → [IMPORT] Move rest of dependant files to shared
Assignee | ||
Comment 2•11 years ago
|
||
Reviews for FTU, Contacts, and locales (I'm not sure if I need l10n revision for this, not a string change, but moving some strings to different files).
Attachment #8403350 -
Flags: review?(l10n)
Attachment #8403350 -
Flags: review?(jmcf)
Attachment #8403350 -
Flags: review?(francisco.jordano)
Comment 3•11 years ago
|
||
Comment on attachment 8403350 [details] [review]
Link to PR - https://github.com/mozilla-b2g/gaia/pull/18083
This should work for l10n seamlessly. I don't think we need to invest deeply into the 5 strings here in terms of string re-use, too.
Moving this to a f+, as that's more of what I did.
Attachment #8403350 -
Flags: review?(l10n) → feedback+
Comment 4•11 years ago
|
||
Comment on attachment 8403350 [details] [review]
Link to PR - https://github.com/mozilla-b2g/gaia/pull/18083
Left some comments on github, not serious but would like to have some clarification on them.
Looking good to me, please flag me once github comments are addressed.
Thanks!
Attachment #8403350 -
Flags: review?(francisco.jordano)
Assignee | ||
Updated•11 years ago
|
Attachment #8403350 -
Flags: review?(francisco.jordano)
Comment 5•11 years ago
|
||
Comment on attachment 8403350 [details] [review]
Link to PR - https://github.com/mozilla-b2g/gaia/pull/18083
Thanks for the clarifications, looking good to me the FTU part, just take a look to Jose's comment about linking.
r+plusing.
Merge once other comments are fixed and travis is green ;)
Thanks Fernando!
Attachment #8403350 -
Flags: review?(francisco.jordano) → review+
Comment 6•11 years ago
|
||
Comment on attachment 8403350 [details] [review]
Link to PR - https://github.com/mozilla-b2g/gaia/pull/18083
thanks Fernando, we can land
Attachment #8403350 -
Flags: review?(jmcf) → review+
Comment 7•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Target Milestone: --- → 1.4 S5 (11apr)
Updated•11 years ago
|
status-b2g-v2.0:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•