Problem/Motivation

We need to verify "/lingotek/notify" is never cached by Drupal

Proposed resolution

Add a test.

Remaining tasks

Patch

User interface changes

None

API changes

None

Data model changes

None

Release notes snippet

None

CommentFileSizeAuthor
#4 3092829-lingotek-notify-not-cached-test-4.patch5.42 KBpenyaskito
#3 3092829-lingotek-notify-not-cached-test-2.patch4.75 KBpenyaskito

Comments

penyaskito created an issue. See original summary.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Active » Needs review

Added patch with test.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
StatusFileSize
new 3092829-lingotek-notify-not-cached-test-2.patch4.75 KB

Forgot the patch attachment

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
StatusFileSize
new 3092829-lingotek-notify-not-cached-test-4.patch5.42 KB

Ensure cache max-age is 0 just in case.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Tests passed as expected.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Reviewed & tested by the community » Fixed

Committed e12bd71 and pushed to 8.x-2.x. Thanks!

  • penyaskito committed e12bd71 on 8.x-2.x 
    Issue #3092829 by penyaskito: Add test to verify "lingotek/notify" is...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.