search

divio / aldryn-newsblog

A combined news/weblog application for Aldryn and django CMS – part of the Essential Addons.
https://marketplace.django-cms.org/en/addons/browse/aldryn-newsblog/
Other
67 stars 118 forks source link

Enabled configurable plugin caching for CMS 3.3.0+ installations #407

Closed mkoistinen closed 8 years ago

mkoistinen commented 8 years ago

This PR adds an extra "Cache Duration" field, and uses it to set caching for "time-sensitive" addons if and only if the project is running on CMS 3.3.0+.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-1.0%) to 89.82% when pulling 35b371327470a15aac99907b2da6cea01e3e9d64 on mkoistinen:feature/conditionally_enable_caching into 7092ad87c87070e4a48b013c1f6a6bfda6cd124e on aldryn:master.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-0.3%) to 90.489% when pulling 30f79c491c8eaf3f43f27333e84f32c49acd6078 on mkoistinen:feature/conditionally_enable_caching into 7092ad87c87070e4a48b013c1f6a6bfda6cd124e on aldryn:master.

GaretJax commented 8 years ago

Left some minor comments, but LGTM

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-0.2%) to 90.559% when pulling 886b0f0a5f507a05f04ff9e643d9915c2be18349 on mkoistinen:feature/conditionally_enable_caching into 7092ad87c87070e4a48b013c1f6a6bfda6cd124e on aldryn:master.