Contrib.social

Change cache to use contexts rather than max-age

Open on Drupal.org →
Created on 23 October 2023, 8 months ago
Updated 19 December 2023, 6 months ago

Problem/Motivation

We noticed that this module won't cache properly, due to being explicitly set, we would like all our pages to
be cached as much as possible.

Steps to reproduce

Install the module, drupal/cache_review
look at page, and the language switcher will have a Red N, which has cache options matched to auto_placeholder_conditions

Proposed resolution

Change from max-age: 0 to use language contexts

🐛 Bug report
Status

Needs work

Version

2.1

Component

Code

Created by

🇳🇿New Zealand atowl

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

Comments & Activities

  • Issue created by @atowl
  • Comment 8 months ago →
    🇳🇿New Zealand atowl
  • Status changed to Needs work 6 months ago
  • Comment 6 months ago →
    🇨🇦Canada SKAUGHT

    is 'language' the only context you are saying should be cached for? myself, I understand the purpose of a language block Not cacheing at all, as other pages maybe updated (results in url-alais change), published/unpublished.. etc. i suspect language alone is lacking..
    -test cases.

contrib.social Blog FAQ Discussions
Production build 0.69.0 2024