search

opral / inlang-sdk

0 stars 0 forks source link

use `null` instead of `undefined` for db schema #141

Closed samuelstroschein closed 2 months ago

samuelstroschein commented 2 months ago

Context

https://github.com/kysely-org/kysely/issues/27#issuecomment-986076120

Proposal

Define database schema types as null | T instead of undefined | T to avoid bugs.

samuelstroschein commented 2 months ago

Was not needed because the inlang schema has no nullable/optional props except for an Expression which is not reflected in the database.