Closed
Bug 1140959
Opened 10 years ago
Closed 10 years ago
[Stingray][Component] Move menu-group to gaia-components repository
Categories
(Firefox OS Graveyard :: Gaia::TV, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rexboy, Unassigned)
References
Details
Attachments
(2 files)
+++ This bug was initially created as a clone of Bug #1140958 +++
We are going to move menu-group component:
https://github.com/smart-components/menu-group
to gaia-components.
I think what we need to do are:
1. open a menu-group repo in gaia-components;
2. Review and check-in menu-group there
(Pull request 1: to gaia-components/menu-group);
3. Make apps that is using menu-group point to gaia-components
(Pull request 2: to gaia).
Reporter | ||
Updated•10 years ago
|
Blocks: tv-gaia-components
Reporter | ||
Comment 1•10 years ago
|
||
https://github.com/gaia-components/menu-group
I've pushed smart-components/menu-group to gaia-components/menu-group.
Now we need to modify bower.json, add readme, and review them.
Reporter | ||
Comment 2•10 years ago
|
||
This is patch for gaia-components repository.
After it got updated we need to update for Gaia repository too.
Tzu-lin may you help review this patch?
Attachment #8576570 -
Flags: review?(tzhuang)
Comment 3•10 years ago
|
||
Comment on attachment 8576570 [details]
patch for gaia-components
Looks good to me, thanks
Attachment #8576570 -
Flags: review?(tzhuang) → review+
Reporter | ||
Comment 4•10 years ago
|
||
Commit for gaia-components:
https://github.com/gaia-components/menu-group/commit/088b750c3fd32c898f2fc30125a41f59eaee1257
And let's patch Gaia master.
I think put them both in this patch should be more clear so let's just do it in this patch.
Reporter | ||
Comment 5•10 years ago
|
||
Tzu-lin: this is Gaia side patch (1 of 3 commits in a pull request)
may you help review this bug?
Attachment #8577122 -
Flags: review?(tzhuang)
Comment 6•10 years ago
|
||
Comment on attachment 8577122 [details]
patch
Looks good to me. Thanks
Attachment #8577122 -
Flags: review?(tzhuang) → review+
Reporter | ||
Comment 7•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•