Closed
Bug 1194092
Opened 9 years ago
Closed 9 years ago
[browser] Firefox Account sync history
Categories
(Firefox OS Graveyard :: Gaia::TV, defect, P1)
Tracking
(feature-b2g:2.5+)
People
(Reporter: yifan, Assigned: danhuang)
References
Details
(Whiteboard: [ft:conndevices][partner-blocker][partner-cherry-pick])
Attachments
(1 file)
Implement Firefox Account sync all data in TV browser.
Reporter | ||
Updated•9 years ago
|
Summary: [browser] Firefox Account sync all → [browser] Firefox Account sync history
Reporter | ||
Updated•9 years ago
|
Depends on: TV_FxAccount
Reporter | ||
Updated•9 years ago
|
Blocks: TV_FxAccount
No longer depends on: TV_FxAccount
Updated•9 years ago
|
feature-b2g: --- → 2.5+
Whiteboard: [ft:conndevices][partner-blocker]
Updated•9 years ago
|
Assignee: nobody → dhuang
Comment 1•9 years ago
|
||
Hi Dan,
Could you provide some feedback about the status of this bug?
Thanks!
Flags: needinfo?(dhuang)
Assignee | ||
Comment 2•9 years ago
|
||
Hi Josh,
I am working on the other bug(Bug 1202376), and just send for review.
These bug would use the same render logic which implement in bug 1202376.
I would start working on this bug after bug 1202376 resolved.
Flags: needinfo?(dhuang)
Updated•9 years ago
|
Target Milestone: --- → FxOS-S10 (30Oct)
Updated•9 years ago
|
Assignee: dhuang → nobody
Updated•9 years ago
|
Priority: -- → P1
Reporter | ||
Updated•9 years ago
|
Assignee: nobody → yliao
Assignee | ||
Updated•9 years ago
|
Assignee: yliao → dhuang
Comment 3•9 years ago
|
||
Assignee | ||
Comment 4•9 years ago
|
||
Comment on attachment 8679826 [details] [review]
[gaia] danhuang1202:1194092_fx_sync_history > mozilla-b2g:master
Please me to review. This patch add historyStore.js for maintain history cache, and delete un-use code for list-dialog. Thanks.
Attachment #8679826 -
Flags: review?(yliao)
Reporter | ||
Comment 5•9 years ago
|
||
Comment on attachment 8679826 [details] [review]
[gaia] danhuang1202:1194092_fx_sync_history > mozilla-b2g:master
The cursor for bookmark and history list is missing. Please file a follow up bug for it. Please see the comments on Github, thanks!
Attachment #8679826 -
Flags: review?(yliao) → review-
Assignee | ||
Comment 6•9 years ago
|
||
Comment on attachment 8679826 [details] [review]
[gaia] danhuang1202:1194092_fx_sync_history > mozilla-b2g:master
Please help to review again. This patch fix the comments you mentioned. Thanks.
Attachment #8679826 -
Flags: review- → review?(yliao)
Assignee | ||
Comment 7•9 years ago
|
||
(In reply to yifan [:yifan][:yliao] from comment #5)
> Comment on attachment 8679826 [details] [review]
> [gaia] danhuang1202:1194092_fx_sync_history > mozilla-b2g:master
>
> The cursor for bookmark and history list is missing. Please file a follow up
> bug for it. Please see the comments on Github, thanks!
The follow up bug 1219564
Reporter | ||
Updated•9 years ago
|
Attachment #8679826 -
Flags: review?(yliao) → review+
Assignee | ||
Comment 8•9 years ago
|
||
Thanks for the review !
merge in master:
https://github.com/mozilla-b2g/gaia/commit/fd0cc6855bd6da560ae48989fb207f76394a74c7
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•9 years ago
|
Whiteboard: [ft:conndevices][partner-blocker] → [ft:conndevices][partner-blocker][partner-cherry-pick]
You need to log in
before you can comment on or make changes to this bug.
Description
•