[SUGGESTION] Does EntityId need to be non-nullable?

[ricochetbrown]

ngrx-toolkit/libs/ngrx-toolkit/src/lib/with-data-service.ts

Line 10 in 92a47ef

export type Entity = { id: EntityId };

On create, it would be better for this to be null so we can do upserts with updateAll. An entityId doesn't exist yet in those cases but we are forced to pass one in.

[ricochetbrown]

We would prefer export type Entity = { id?: EntityId };