Matti Salokangas
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...