search

ArkerLabs / redisk

TypeScript ORM for Redis.
33 stars 8 forks source link

The error message on redisk.save is misleading #5

Closed EduardoSimon closed 4 years ago

EduardoSimon commented 4 years ago

When a field of one event is null or undefined in an updater, due to a developer error, the output from the console is misleading. The error should point right to the updater and the field that was null || undefined.

In the following case the updater had the event.modified to null. Causing an error.

image

And the output from the console was:

image

Nytyr commented 4 years ago

Can you send me your Entity? So I can reproduce the bug.

EduardoSimon commented 4 years ago

In the previous example, the event.email field was null. So, in order to recreate it try to force accesing a null field of the event and persist it in a redis entity.

Nytyr commented 4 years ago

I need the EmployeeViewModel implementation, I'm trying to update other entities with Date fields and it's working fine.

EduardoSimon commented 4 years ago

Try to update a primary key field with null, thats what I think I am doing wrong

Nytyr commented 4 years ago

Closing, can't reproduce.