Remove Product Announcements

RESOLVED FIXED in Firefox 33

Status

Android Background Services
Product Announcements
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: rnewman, Assigned: rnewman)

Tracking

(Blocks: 1 bug)

Firefox 34
Firefox 34
All
Android
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox33 fixed, firefox34 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Wake up! It's time to die!
(Assignee)

Updated

4 years ago
Depends on: 1045087
woot!
(Assignee)

Comment 2

4 years ago
Created attachment 8463548 [details] [diff] [review]
Remove main Product Announcements code. v1

This is based on <https://github.com/mozilla-services/android-sync/pull/473>. It's here as an hg patch so that file removals are correct.

Note that the patch in Bug 1045087 is necessary for complete function.
Attachment #8463548 - Flags: review?(michael.l.comella)
(Assignee)

Updated

4 years ago
Blocks: 1045278
Comment on attachment 8463548 [details] [diff] [review]
Remove main Product Announcements code. v1

Review of attachment 8463548 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm w/ nit on gh.
Attachment #8463548 - Flags: review?(michael.l.comella) → review+
https://hg.mozilla.org/mozilla-central/rev/2175dc19592a
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 34
(Assignee)

Comment 6

4 years ago
Comment on attachment 8463548 [details] [diff] [review]
Remove main Product Announcements code. v1

We'd like this on Beta. Might as well kill it early.

Approval Request Comment
[Feature/regressing bug #]:
  N/A
[User impact if declined]:
  N/A
[Describe test coverage new/current, TBPL]:
  N/A -- feature removal.
[Risks and why]: 
  Feature removal. Low risk.
[String/UUID change made/needed]:
  None.
Attachment #8463548 - Flags: approval-mozilla-beta?
Comment on attachment 8463548 [details] [diff] [review]
Remove main Product Announcements code. v1

Although a big patch, this is a pretty small change. I agree with the risk assessment and the upside is that we can discontinue the server side of this feature 6 weeks earlier. Beta+
Attachment #8463548 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
https://hg.mozilla.org/releases/mozilla-beta/rev/776ddfd41f21
status-firefox33: --- → fixed
status-firefox34: --- → fixed
(Assignee)

Comment 10

4 years ago
ni on me for a non-Ryan landing :D
Flags: needinfo?(rnewman)
(Assignee)

Comment 11

4 years ago
Ryan et al: the reason this didn't land cleanly is that it depends on Bug 1045087, which removes the integration points for this code. I flagged that for beta; it's also a safe change.
https://hg.mozilla.org/releases/mozilla-beta/rev/d5ed7dd8f996
status-firefox33: affected → fixed
Flags: needinfo?(rnewman)
You need to log in before you can comment on or make changes to this bug.