Remove clear cache button #1275
Conversation
|
We built our own clear cache button, but only developers used it. Eventually, we just built it into the deploy process. 👍 for removal. |
|
Another point I should mention is that stores with heavy custom caching (which is who I assume this is designed for) are likely to need more than just |
|
👍 for removing button. Only developers should need to do platform operations like clearing the Rails cache, and we have access to the Rails console for this. Too dangerous to give non-developers things they can mess things up with (we have enough problems with them creating Promos that break our checkout!) |
|
👍 |
Our existing "General Settings" page has a "Clear cache" button. On stores I've worked on I wouldn't want this exposed to admins. We should (and afaik we do) expire caches every time the underlying DB object is changed. This is motivated partly because I want to replace the GeneralSettingsController with a StoreController, and this is the only thing on that page that doesn't directly relate to a store.
jhawthorn
force-pushed
the
remove_clear_cache
branch
from
49d900a to
9ead23f
Compare
|
Rebased and added note to changelog |
Our current "General Settings" page has a "Clear cache" button.
On stores I've worked on I wouldn't want this exposed to admins. We should (and afaik we do) expire caches every time the underlying DB object is changed.
This is motivated partly because I want to replace the
GeneralSettingsControllerwith aStoreController, and this is the only thing on that page that doesn't directly relate to a store.Though this should be trivial to replace in one's own store, I'd love to hear if anyone considers this important in their admin or would be inconvenienced by it being removed.