Marc Roig
Marc Roig
@derrickmehaffy you are right My first guess was to first delete the components before the entity. But It makes more sense just having a cascade delete, I will discuss it...
About the components not being removed from the database, @derrickmehaffy mentions in the [issue](https://github.com/strapi/strapi/issues/14039#issuecomment-1209570377) that we could modify the foreign key constrain to enable cascade delete. @petersg83 we could have...
> et's talk about the cascade delete, that may be an option. What do you think to separate this in 2 PR? One for the webhook and one for the...
@petersg83 I see an e2e failing with my current change: `packages/core/strapi/tests/lifecycles/before-delete.test.e2e.js` Exactly at line 94. From there, I saw a PR made by @stafyniaksacha some months ago: https://github.com/strapi/strapi/pull/12653 He changed...
Hello @WalkingPizza , thank you for taking the time to work & fix this! Your idea was excelent and I think I have a way to improve it further 😬...
Another thing to take into consideration... In `packages/core/upload/server/services/upload.js` L88, the file.size is transformed to KB. In the docs, they say the `sizeLimit` argument should be in bytes, but seeing this,...
that is ok with me too! 🙂
Hello @WalkingPizza . I had a discussion today with @petersg83 and @alexandrebodin about this 😃 About the file size being in KB. We concluded this should stay like this to...
I like the idea, but it feels weird to have an argument in the provider options that does not get to the provider itself. `sizeLimit` argument already existed and was...
I would say to keep it like this, as all our supported providers give no errors. Also, passing the sizeLimit in the provider config was the default configuration before, so...