Bug 1524322 Comment 9 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

## Beta/Release Uplift Approval Request

### Feature/Bug causing the regression

Bug 9042224

### User impact if declined

This component is used in the upcoming Pocket New Tab experience and will be used to deliver legal disclaimers to users.

### Is this code covered by automated tests?

No

### Has the fix been verified in Nightly?

Yes

### Needs manual test from QE?

Yes

### If yes, steps to reproduce

1. Open `about:config`
2. Change `browser.newtabpage.activity-stream.discoverystream.config` to `{"enabled":true,"show_spocs":true,"layout_endpoint":"https://getpocket.com/v3/newtab/layout?version=1&consumer_key=40249-e88c401e1b1f2242d9e441c4&layout_variant=dev-test-all"}`
3. Open a new tab
4. Confirm that there's a component matching this comp: https://bug1524322.bmoattachments.org/attachment.cgi?id=9042229

### List of other uplifts needed

None

### Risk to taking this patch

Low

### Why is the change risky/not risky? (and alternatives if risky)

This component is self-contained and has no external dependencies.

### String changes made/needed
## Beta/Release Uplift Approval Request

### Feature/Bug causing the regression

Bug 1524322

### User impact if declined

This component is used in the upcoming Pocket New Tab experience and will be used to deliver legal disclaimers to users.

### Is this code covered by automated tests?

No

### Has the fix been verified in Nightly?

Yes

### Needs manual test from QE?

Yes

### If yes, steps to reproduce

1. Open `about:config`
2. Change `browser.newtabpage.activity-stream.discoverystream.config` to `{"enabled":true,"show_spocs":true,"layout_endpoint":"https://getpocket.com/v3/newtab/layout?version=1&consumer_key=40249-e88c401e1b1f2242d9e441c4&layout_variant=dev-test-all"}`
3. Open a new tab
4. Confirm that there's a component matching this comp: https://bug1524322.bmoattachments.org/attachment.cgi?id=9042229

### List of other uplifts needed

None

### Risk to taking this patch

Low

### Why is the change risky/not risky? (and alternatives if risky)

This component is self-contained and has no external dependencies.

### String changes made/needed

Back to Bug 1524322 Comment 9