Allow optional fields to be null

[honzajerabek]

Hi there, currently the fields that are nullable are only noted as ?, which allows them to be omitted or undefined. This however doesn't allow them to be set null explicitly.

This is needed mainly when updating to set a field to null. Omitting the field causes no change, passing undefined throws an error in sequelize.

We should be able to call model.update({ name: null })

[schtomas]

I have the same issue. If i generate models from database, all nullable fields appear as possibly undifined and it make problems if i want to insert null or select values where is not null it shows type errors. I had to fix it manualy in code and it looks ok now, but if i generate again models i have to fix it again. Will be some fix for this please?

Solution above it solves maybe problem but what if node modules are installed again? We are useing docker and scripts always install modules from scratch so you have to update again this file autogenerator.ts manualy before generating modelas

[aeharake]

We should be able to call model.update({ name: null })

Who said you can't do that? You can definitely do it! You're missing something for sure, please open an issue and provide an example of your failing code.

[bossvip]

We should be able to call model.update({ name: null })

Who said you can't do that? You can definitely do it! You're missing something for sure, please open an issue and provide an example of your failing code.

https://stackoverflow.com/questions/57243477/can-an-optional-parameter-be-null-in-typescript

"strictNullChecks": true is the default in frameworks like Nextjs. You can check the inferred type for the below code in VSCode.

It is possible to add @ts-ignore and pass null values, but that defeats the point of typescript. I had to update the generated model by hand. It is 2024, why is this PR not merged yet?

• null not allowed

interface TestProps {
  myVal?: string
 }

• null allowed

interface TestProps {
  myVal?: string | null
 }