-
-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migration of REST Cache to Strapi 5, please read and give your thoughts #97
Comments
@derrickmehaffy I would like to help wherever I could to get this plugin v5 released, Is there any specific action item I could pick ? |
Thanks for your work on this plugin. Really appreciated that for v4. Thats a pretty ambitious plan for the rewrite. Most of these suggestions look excellent. The only use case I currently use Strapi for is to serve pretty static content that allows editors to manage the content easily. The most important missing feature in v4 is the lack of control over the injection of the middleware. The documentation was a bit unfocused but contained all relevant info in my case. Most of the other features proposed here are nice to have, but absolutely not required. Cache busting always worked well enough in v4 and I rarely (maybe never) ran into issues where I wanted to debug the cache. I setup the cache and it 'just worked'. So from my perspective, I would focus on improving documentation and allowing control over the order of middleware injection. |
Refactoring the way the middleware gets injected & improving documentation sounds like necessary changes. But I need to agree about the other features. They're not really necessary and most of them should be non-breaking which means you could ship them some time later. For me the most important point is getting this plugin working with Strapi 5 fast. This plugin is the only major thing blocking us from upgrading right now. |
This plugin is the main obstacle preventing our upgrade to Strapi 5. It must be compatible with the new version as soon as possible. Thank you |
Thank you all for your inputs. Let me chat with the other maintainers and see if we can come up with a short and long term plan. |
Yes, we are facing the same problem. |
+1 |
1 similar comment
+1 |
Hello all o/
I am Derrick, a Support Engineering Manager at Strapi and also one of the co-maintainers of this plugin. With the release of Strapi 5 I will be working with @stafyniaksacha and @Boegie19 to do some heavy rewrites to this plugin to update it to Strapi 5. There almost certainly will be some breaking changes but with that will ideally bring some new features hopefully.
Below are some things I would like to tackle during the rewrite, albeit we don't know how long this rewrite is going to take. Regardless I'll detail some of my thoughts below but would also like to hear yours.
strapi-plugin-rest-cache
->@strapi-community/plugin-rest-cache
strapi-provider-rest-cache-couchbase
->@strapi-community/provider-rest-cache-couchbase
strapi-provider-rest-cache-memory
->@strapi-community/provider-rest-cache-memory
strapi-provider-rest-cache-redis
->@strapi-community/provider-rest-cache-redis
And I think that's about all I had in mind so far but I'm open to hearing more community suggestions
The text was updated successfully, but these errors were encountered: