I recommend using undefined instead of null here and everywhere else in the mapping classes. Just because that's what they are everywhere else.
|
get region(): string | null { |
|
return this.singularNullable(VCARD.region, ValueMapping.literalToString) ?? null; |
To be clear, I don't think that null is wrong. I'm just trying to keep everything as similar as possible. But if you have any compelling reason that do keep it.
I recommend using
undefinedinstead ofnullhere and everywhere else in the mapping classes. Just because that's what they are everywhere else.volunteering-demo/app/lib/class/Agent.ts
Lines 25 to 26 in 9fde751
To be clear, I don't think that
nullis wrong. I'm just trying to keep everything as similar as possible. But if you have any compelling reason that do keep it.