Ben Irvin

Results 53 comments of Ben Irvin

> Should we move "loadPkg" and "logger" to strapi/utils ? Probably beyond the scope of this PR, but yeah, I think we need a generic logger to use for all...

> I'm a bit surprised, Does this work everywhere else ? I seem to remember I had to drop the index on foreign keys before dropping them to avoid errors...

It turns out only mysql (and nobody else) drops the index along with the fk when you drop a fk. So I added a check that for mysql only, we...

> I did need to try starting the app twice after this change on the first start I got this error I wonder if that's an issue upgrading from 5.0.0-beta.1...

@MarionLemaire I improved the utility functions a bit and pushed the changes we did this afternoon, they should all be passing now!

I think this will be fine generally, but I need to determine that there aren't additional security implications. However, we have a freeze on Strapi 4 (the `develop` branch) for...

Hello, thank you for your PR! Sorry for the delay in getting to this. Could you please update the branch to target `develop` instead of `main` ?

@Convly To pre-empt your comments you mentioned earlier, I think opening a local file provider is actually the best solution. Otherwise we have to write additional code to check the...

Closing for now since we have a different solution for the main issue.

The prettier cache is stored in the node_modules/.cache/prettier folder The eslint cache is stored in the strapi root as .eslintcache, which is why I added that to .gitignore. In theory...