Closed Bug 1384021 Opened 4 years ago Closed 4 years ago

Enable ActivityStream for all 57+ users

Categories

(Firefox for Android Graveyard :: General, enhancement, P1)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
All
Android
Type:
enhancement

Tracking

(firefox57 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 57
Tracking Flags:
Tracking Status
firefox57 --- fixed

People

(Reporter: sebastian, Assigned: sebastian)

References

Details

(Whiteboard: [MobileAS] 1.26)

Attachments

(2 files)

Bug 1384021 - Enable Activity Stream for all users and remove experiment/setting.
59 bytes, text/x-review-board-request
mcomella
: review+
Details
Bug 1384021 - ActivityStream: Remove outdated constants.
59 bytes, text/x-review-board-request
mcomella
: review+
Details 
I talked to Maria and we want to go from 50% to 100% in order to get more feedback (and possibly crash reports).
Note that we are still going to show the "Experimental features" setting so that users can opt-out (at least in Nightly 56).
Updated the staging bucket.

@Grisha: Can you review/approve my changes in the Kinto admin?
Flags: needinfo?(gkruglov)
Whiteboard: [MobileAS] → [MobileAS] 1.26
Iteration: 1.26 → 1.27
Now that we are on 57 I wonder whether we should just turn it on - without the experiment.
Summary: Enable ActivityStream for all Nightly users → Enable ActivityStream for all 57+ users
Do it! And I just got my Viscosity VPN license...
Flags: needinfo?(gkruglov)
(In reply to :Grisha Kruglov from comment #6)
> Do it! And I just got my Viscosity VPN license...

You can still press that button .. will take some time until patches land and are deployed .. :)
Comment on attachment 8892984 [details]
Bug 1384021 - Enable Activity Stream for all users and remove experiment/setting.

https://reviewboard.mozilla.org/r/163982/#review169514

I wonder if there are any UI tests that need updating?
Attachment #8892984 - Flags: review?(michael.l.comella) → review+
Comment on attachment 8892985 [details]
Bug 1384021 - ActivityStream: Remove outdated constants.

https://reviewboard.mozilla.org/r/163988/#review169516
Attachment #8892985 - Flags: review?(michael.l.comella) → review+
(In reply to Sebastian Kaspari (:sebastian) from comment #7)
> (In reply to :Grisha Kruglov from comment #6)
> > Do it! And I just got my Viscosity VPN license...
> 
> You can still press that button .. will take some time until patches land
> and are deployed .. :)

Jason just approved it so that we can move forward with Bug 1386172. Thanks all.
Pushed by s.kaspari@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/57e3ad20c661
Enable Activity Stream for all users and remove experiment/setting. r=mcomella
https://hg.mozilla.org/integration/autoland/rev/be21172b5b5c
ActivityStream: Remove outdated constants. r=mcomella
https://hg.mozilla.org/mozilla-central/rev/57e3ad20c661
https://hg.mozilla.org/mozilla-central/rev/be21172b5b5c
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
One of the reasons for turning this on was to get more feedback. Where is the proper place to give feedback?
(In reply to Greg K Nicholson [:gkn] from comment #17)
> One of the reasons for turning this on was to get more feedback. Where is
> the proper place to give feedback?

I think the options, in order of preference, are:
- File a new bug in the Firefox for Android product (make sure you include "Activity Stream" or "AS" in the bug title)
- Create a discussion on mobile-firefox-dev@ [1]
- Talk to us on #mobile (I think this option is most likely to be overlooked unless you ping one of the core AS devs directly - myself, :liuche, or :sebastian)

Thanks in advance for any feedback! :)

[1]: https://mail.mozilla.org/listinfo/mobile-firefox-dev
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.