Closed
Bug 1488927
Opened 6 years ago
Closed 6 years ago
Clicking remove on a study in about:studies doesn't work
Categories
(Firefox :: Normandy Client, defect)
Firefox
Normandy Client
Tracking
()
VERIFIED
FIXED
Firefox 64
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox62 | --- | unaffected |
firefox63 | --- | verified |
firefox64 | --- | verified |
People
(Reporter: mythmon, Assigned: mythmon)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
aswan
:
review+
pascalc
:
approval-mozilla-beta+
|
Details |
When clicking on remove in about:studies, the study is not removed. Instead the an error is thrown: TypeError: AddonStudies.stop is not a function AboutPages.jsm:186:13
I believe this is a regression introduced in bug 1440780, which is in beta 63 now.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → mcooper
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•6 years ago
|
||
Updated•6 years ago
|
Keywords: regression
Updated•6 years ago
|
Flags: qe-verify+
Comment 2•6 years ago
|
||
Comment on attachment 9006715 [details]
Bug 1488927 - Make about:studies "remove" button work again
Andrew Swan [:aswan] has approved the revision.
Attachment #9006715 -
Flags: review+
Pushed by mcooper@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/94a9c739e329
Make about:studies "remove" button work again r=aswan
Comment 4•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Comment 5•6 years ago
|
||
(In reply to Michael Cooper [:mythmon] from comment #0)
> I believe this is a regression introduced in bug 1440780, which is in beta
> 63 now.
Michael, could you backport your patch to the 63 branch? Thanks
Flags: needinfo?(mcooper)
Assignee | ||
Comment 6•6 years ago
|
||
Comment on attachment 9006715 [details]
Bug 1488927 - Make about:studies "remove" button work again
Approval Request Comment
[Feature/Bug causing the regression]: 1440780
[User impact if declined]: Users are unable to unenroll from studies in about:studies
[Is this code covered by automated tests?]: No (see bug 1442712)
[Has the fix been verified in Nightly?]: No
[Needs manual test from QE? If yes, steps to reproduce]: Yes
1. Enroll in an add-on study (Using the Normandy Devtools [0] is recommend)
2. Open about:studies
3. Click "Remove"
Expected: the study is removed, the button changes state
Actual: The button does not change state and the study is not removed.
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: A simple change that is easy to test.
[String changes made/needed]: None
Flags: needinfo?(mcooper)
Attachment #9006715 -
Flags: approval-mozilla-beta?
Comment 7•6 years ago
|
||
Comment on attachment 9006715 [details]
Bug 1488927 - Make about:studies "remove" button work again
Fix for a functional regression on the about:studies page on 63 while we have several upcoming Shield studies, uplift approved for 63 beta 6
Attachment #9006715 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 8•6 years ago
|
||
Adrian, would your team have the cycles to test the fix first on Nighty and then on Beta this week? Thanks
Flags: needinfo?(adrian.florinescu)
Comment 9•6 years ago
|
||
(In reply to Pascal Chevrel:pascalc from comment #8)
> Adrian, would your team have the cycles to test the fix first on Nighty and
> then on Beta this week? Thanks
For this fix alone, there is no problem verifying it this week.
I'm concerned though if this regression isn't part of a bigger issue. I'll ping :mythmon to follow up on that.
Updated•6 years ago
|
Blocks: 1440780
status-firefox62:
--- → unaffected
status-firefox63:
--- → affected
status-firefox-esr60:
--- → unaffected
No longer depends on: 1440780
Assignee | ||
Comment 10•6 years ago
|
||
The larger issue here was that the tests for about:studies were disabled, so automated tests didn't catch this. Tests have now been fixed and re-enabled, so there isn't a larger issue related to this for Adrian or his team to investigate.
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(adrian.florinescu)
Comment 11•6 years ago
|
||
bugherder uplift |
Comment 12•6 years ago
|
||
Reproduced the issue on 63 Beta5;
Verified as fixed on Windows 10, Windows 8.1:
64.0a1 20180913100107
63.0b6 20180913141435
Although due to comment 10, the manual verification wasn't necesary, I've verified this fix for good practice, removing qe verify flags.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•