Open dimitrisganotis opened 1 month ago
Hi @dimitrisganotis,
Thanks for reaching out! Strapi v5 was recently released, and I plan to wait at least 2 months before updating the plugin to v5 myself. This will allow time for any initial bugs or issues in the new version to be resolved, ensuring a smoother transition.
In the meantime, you're more than welcome to contribute to the project by creating a new branch for Strapi 5 support. Contributions are always appreciated!
Best regards, m7rlin
Doesn't seem to be too challenging, take a look at the one existing fork of this project (cezarvrabie99). The only breaking change is, you need to use a different request parameter with Strapi v5.
I don't understand why team Strapi is blocking this. Looking at the code it isn't too expensive for performance as it should immediately return if the param isn't availabe (see fork code) or the model doesn't match the allowed models. There is still some performance optimization possible for the latter case in the current code - which is worth it, as other models like permission stuff are queried among every request.
Many devs like me use deep (or unpredictably deep) nested structures and I couldn't do my projects without an extension like this...
Some other fix:
- event.params.populate = true
+ event.params.populate = {}
There is a fork on NPM that piggybacks your repo name, which I found super confusing. Is it possible to merge their changes back to this repo and/or take down the package? Not sure if it's legit:
https://www.npmjs.com/package/magictm-strapi-v5-plugin-deep-populate
Is there a target date for the plugin update to support Strapi 5?