Closed
Bug 642664
Opened 13 years ago
Closed 13 years ago
Rip out first start logic
Categories
(Cloud Services :: Share: Firefox Client, defect)
Cloud Services
Share: Firefox Client
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philikon, Assigned: philikon)
Details
Attachments
(1 file)
14.13 KB,
patch
|
mixedpuppy
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•13 years ago
|
||
Now that we're integrated into Firefox, does the first start logic even make sense anymore? Maybe we should morph that bug into "rip out first start logic"? I guess it would be good to promote the feature for users that upgrade from Fx4 to 5, but I feel that's better served with the in-product promotion a la Sync (http://areweprettyyet.com/5/syncPromotion/)
Comment 2•13 years ago
|
||
IMO rip it out, less code is better. Do you want to take this bug?
Assignee | ||
Updated•13 years ago
|
Summary: Put first start logic into install() → Rip out first start logic
Assignee | ||
Comment 4•13 years ago
|
||
Rips out everything related to first runs. Hope I got everything.
Attachment #521988 -
Flags: review?(mixedpuppy)
Assignee | ||
Updated•13 years ago
|
Whiteboard: [ETA 2011-03-25] → [has patch][needs review mixedpuppy]
Comment 5•13 years ago
|
||
Comment on attachment 521988 [details] [diff] [review] v1 cool.
Attachment #521988 -
Flags: review?(mixedpuppy) → review+
Assignee | ||
Comment 6•13 years ago
|
||
https://hg.mozilla.org/users/pweitershausen_mozilla.com/fx-share/rev/05f0b6108913
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [has patch][needs review mixedpuppy]
You need to log in
before you can comment on or make changes to this bug.
Description
•