Results 24 comments of Matti Salokangas

@andrewplummer I took a stab at fixing this issue. I very well may not have solved it correctly, however, the test I added confirmed that it does fix the issues...

👋I'd like to take a stab at this. @zouxuoz -Do you mind if I PR this?

Apologies, I missed that. Thank you!

I have an open PR that has changes related to this. I can try to fix there, but just waiting on some feedback before I put more time into https://github.com/sequelize/sequelize/pull/11514

Forked it and fixed the warning if you want to use my fork: https://github.com/sturdynut/react-native-easy-toast

@nlf - Did you ever have a chance to take a look at this? Anyway to set `process.env.npm_lifecycle_event` in hooks if it is undefined?

Dug into this a little more and it appears as though the issue is when an array of patches are performed for an array. Here is a failing test: ```...

@papb - Thank you! I've used sequelize on many projects and it is a pleasure to help out. 😃 I've pushed my latest updates based on your feedback. * Returned...

@papb - All good! I just wanted to make sure you knew that it was ready for re-review. Thank you!

@sushantdhiman -Are you referring to where I've renamed a parameter because I added another? An example is where I've renamed "attributes" to "sqlAttributes". I'm happy to make the change I...