Page MenuHomePhabricator

CentralNotice: Use LocalStorage instead of cookies whenever possible, for buckets, impression limiting and large banner limiting
Closed, ResolvedPublic4 Story Points

Description

Buckets, impression limiting and large banner limiting are the three features that we can move from cookies to LocalStorage right away.

As per T131319, for now, we'll leave out the decookieing of GeoIP and banner hiding after close button clicks and donations. (GeoIP is used a lot beyond CentralNotice; changes there should be part of a wider discussion about removing it from CentralNotice. For banners, hiding, we can't easily move to LocalStorage because our current way of hiding banners over multiple sites wouldn't work.)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 13 2016, 10:03 PM
AndyRussG renamed this task from CentralNotice: Use LocalStorage instead of cookies whenever possible to CentralNotice: Use LocalStorage instead of cookies whenever possible, for buckets, impression limiting and large banner limiting.Apr 21 2016, 6:49 PM
AndyRussG updated the task description. (Show Details)

Change 284934 had a related patch set uploaded (by AndyRussG):
[Draft] Use LocalStorage for impression diet

https://gerrit.wikimedia.org/r/284934

Change 285336 had a related patch set uploaded (by AndyRussG):
[WIP] Mixed storage for impression diet and large banner limit

https://gerrit.wikimedia.org/r/285336

Change 284934 abandoned by AndyRussG:
[Draft] Use LocalStorage for impression diet

Reason:
Trying a different approach; see Icd88f7bf9.

https://gerrit.wikimedia.org/r/284934

Change 285571 had a related patch set uploaded (by AndyRussG):
[WIP] Mixed storage for buckets

https://gerrit.wikimedia.org/r/285571

Change 285336 merged by jenkins-bot:
Mixed storage for impression diet and large banner limit

https://gerrit.wikimedia.org/r/285336

Change 285571 merged by jenkins-bot:
Mixed storage for buckets

https://gerrit.wikimedia.org/r/285571

Change 288143 had a related patch set uploaded (by AndyRussG):
kvStoreMaintenance: check for and maybe remove fallback cookies

https://gerrit.wikimedia.org/r/288143

Change 288143 merged by jenkins-bot:
kvStoreMaintenance: check for and maybe remove fallback cookies

https://gerrit.wikimedia.org/r/288143

AndyRussG closed this task as Resolved.