search

strapi / foodadvisor

🥘 THE Strapi demo application
MIT License
1.09k stars 438 forks source link

Remove populate deep plugin #135

Open derrickmehaffy opened 9 months ago

derrickmehaffy commented 9 months ago

We have officially declared the populate deep plugin as not recommended and we should not use it in any public example application and have removed it from our marketplace due the performance issues it causes.

This plugin should be removed from the example and specific population should be used following our documentation and best practices.

derrickmehaffy commented 9 months ago

Dependency:

https://github.com/strapi/foodadvisor/blob/dc5b05429964ade1446a579396fe53183f6de07d/api/package.json#L25

StrapiBartosh commented 9 months ago

The to-do plugin should also be removed as it is not compatible with node 18

https://github.com/strapi/foodadvisor/blob/dc5b05429964ade1446a579396fe53183f6de07d/api/package.json#L26

remidej commented 9 months ago

@StrapiBartosh that should be fixed if we bump the version to 0.0.10

Mcastres commented 9 months ago

I created a PR for that: https://github.com/strapi/foodadvisor/pull/137