Add support page caching (Boost, Varnish)

Created on 10 October 2014, over 10 years ago
Updated 6 August 2024, 5 months ago

Problem/Motivation

Our site uses Varnish for page caching, and this module would only worked for authenticated users, because it relies on PHP sessions. We needed to re-write parts of this module to use Javascript and cookies to determine when to display pop-up, not backend PHP.

Proposed resolution

Instead of relying on backend PHP, use JS and cookies to determine when to show pop-up. This ensures that all clients get the functionality, even if they have been served the page from external cache, like Varnish or boost.

See attached patch.
As much as I had time, I also cleaned up code for Drupal coding standards.

Remaining tasks

I've tested minimally using Chrome and Firefox on Mac. The JS should be IE6+ compatible, but I'm not able to test that right now.

User interface changes

n/a

API changes

n/a

✨ Feature request
Status

Needs work

Version

1.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States krisahil

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024