Closed Bug 1516041 Opened 5 years ago Closed 5 years ago

Clear cache / force refresh if pocket newtab endpoint changes

Categories

(Firefox :: New Tab Page, enhancement, P1)

Product:
Firefox
Component:
New Tab Page
Type:
enhancement

Tracking

()

Status:
VERIFIED FIXED
Milestone:
Firefox 66
Iteration:
66.3 - Jan 7 - 20
Tracking Flags:
Tracking Status
firefox66 --- fixed

People

(Reporter: k88hudson, Assigned: k88hudson)

References

Details

When our pocket layout pref (discoverystream.config) is updated, we do cause a refresh however we do not automatically invalidate the cache. We should probably do that if the endpoint value changes.
Note that this is a follow-up to Bug 1516008
Assignee: nobody → khudson
Iteration: --- → 66.2 - Dec 24 - Jan 6
Priority: -- → P1
Commit pushed to master at https://github.com/mozilla/activity-stream

https://github.com/mozilla/activity-stream/commit/6d2e20e9ab47b0c9f336e3c74e9598645182724d
Bug 1516041 - Clear cache if pocket newtab endpoint changes (#4632)

Also fixes issue where cache time check was reversed, adds test
Blocks: 1517867
Iteration: 66.2 - Dec 24 - Jan 6 → 66.3 - Jan 7 - 20

https://hg.mozilla.org/mozilla-central/rev/bd9fccec51d7

Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox66: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
Status: RESOLVED → VERIFIED
Severity: normal → enhancement
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.