Closed Bug 642664 Opened 13 years ago Closed 13 years ago

Rip out first start logic

Categories

(Cloud Services :: Share: Firefox Client, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: philikon, Assigned: philikon)

Details

Attachments

(1 file)

      No description provided.
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/)
IMO rip it out, less code is better.  Do you want to take this bug?
Yep, I can take it if you review :)
Assignee: mixedpuppy → philipp
Summary: Put first start logic into install() → Rip out first start logic
Attached patch v1Splinter Review
Rips out everything related to first runs. Hope I got everything.
Attachment #521988 - Flags: review?(mixedpuppy)
Whiteboard: [ETA 2011-03-25] → [has patch][needs review mixedpuppy]
Comment on attachment 521988 [details] [diff] [review]
v1

cool.
Attachment #521988 - Flags: review?(mixedpuppy) → review+
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.

Attachment

General

Created:
Updated:
Size: