pabsttheater / pabstetix

Issue Tracking for pabsttheater.org
0 stars 0 forks source link

Post Expiry plugin should respect expiration method settings #8

Open sarandi opened 6 years ago

sarandi commented 6 years ago

Setting a slide to expire to draft at D datetime should trigger a cache update, but doesn't. Because of this we've been setting slides to Delete, which isn't ideal, but works.

In other words, if you set a slide to expire at November 21 at 9AM via draft, it stays on the page - but if you leave it as such and just clear the cache, it goes away.

sarandi commented 6 years ago

This is still broken on staging and production. Here's documentation of my testing on staging, using an incognito window, not signed in:

Here's a screenshot showing the slide expiration settings: image

Here are screenshots showing that the slide didn't clear at the set time: image

When I try to flush the cache from WP, I get this error: image

sarandi commented 6 years ago

This looks good on http://rhppabst17stge.wpengine.com, sorry again for the mixup.

sarandi commented 6 years ago

@supergunMKE reported that the "Trash" expiration method had been working, but now appears to be broken after our last (or next to last) code push. The Michelle Wolf slide was set to expire on a set datetime using trash, but didn't do so even after the 10 minute cache max.

etix-aaron commented 5 years ago

@supergunMKE is now setting slides to expire to the trash. will check in next week to see how the function is performing.