statamic / eloquent-driver

Provides support for storing your Statamic data in a database, rather than flat files.
https://statamic.dev/tips/storing-content-in-a-database
MIT License
104 stars 73 forks source link

Change blink handle for 'all' as it clashes with handles of 'all' #136

Closed ryanmitchell closed 1 year ago

ryanmitchell commented 1 year ago

Fixes #135

what-the-diff[bot] commented 1 year ago
what-the-diff[bot] commented 1 year ago
## PR Summary

* **Updated cache key for asset containers**
Changed the cache key for all asset containers from 'eloquent-assetcontainers-all' to 'eloquent-assetcontainers'.
* **Updated cache key for collections**
Changed the cache key for all collections from 'eloquent-collections-all' to 'eloquent-collections'.
* **Updated cache key for fieldsets**
Changed the cache key for all fieldsets from 'eloquent_fieldsets_all' to 'elouqnet_fieldsets'.
* **Updated cache key for global sets**
Changed the cache key for all global sets from 'eloquenrt globals etasl' to 'eolque ntgloblasest'.
jasonvarga commented 1 year ago

Sorry for the delay.

Looks like the conflict only happened for asset containers because the blink key for singular containers used the plural.

e.g. assetcontainers-something

Everywhere else, for the singular, it used the singular.

e.g. collection-something

What you've done is still a good change to make though. 👍