sequelize icon indicating copy to clipboard operation
sequelize copied to clipboard

Update should accept a Partial Model as first parameter

Open victorivens05 opened this issue 5 years ago • 0 comments

The first parameter for update is values: object, where anything can be passed as a parameter, as in the line: https://github.com/types/sequelize/blob/69b371d9dc754d409191556d032dba5aa6f42c53/lib/model.d.ts#L1907

I think it should be values: Partial<M>. That way we could be confident that if we change a parameter in any model, we would see the error, since the value would only accept values present in the model.

victorivens05 avatar Dec 19 '19 17:12 victorivens05