Alyx

Results 453 comments of Alyx

I can confirm that the documentation is wrong. Based on the history, this feature never existed. It looks like the documentation was incorrectly updated based on this issue having been...

I've just ran a test for `individualHooks` and did not see anything wrong. If you're encountering bugs with that option, consider opening its own bug report with a reproducible example...

~~Here is a feature proposal that could resolve this issue: https://github.com/sequelize/sequelize/issues/15493. Feedback welcome :)~~ I actually think this solution is a dead-end. Adding the possibility of listening to database triggers...

Well it does turn out [this feature exists](https://github.com/sequelize/sequelize/blob/340993dc4a220c0cf493bfdb2eec335498e8abf6/packages/core/src/dialects/abstract/query-interface.js#L799-L806C7) after all To be honest I really dislike that it's repurposing the `hooks` option which already has a very different purpose and...

I've updated the link to the relevant piece of code: https://github.com/sequelize/sequelize/blob/340993dc4a220c0cf493bfdb2eec335498e8abf6/packages/core/src/dialects/abstract/query-interface.js#L799-L806C7

Not really, `beforeSave` is called when `save` is called, and upsert does not use `save`, so there is nothing to fix here after all.

It is about matching the method name because the hook lets your modify the option it gives you. If we made upsert arbitrarily call `beforeSave` with its option, you'd get...

None yet but I expect the complete rewrite of WHERE we're doing will fix this, as it's one of its goals (PR https://github.com/sequelize/sequelize/pull/15598)

All errors except one are caused by MariaDB returning JS BigInts instead of strings by default for bigints. It's a good change but we should provide a way to opt...

This is likely implemented by https://github.com/sequelize/sequelize/pull/15598, but a future PR will add a test to make sure