You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
one of the things about the Field types is that the zero value is invalid (that's why there's both a _set and a _null), and you have to use the constructors to create values correctly. using embedding like this would negate that somewhat. it's also not very many lines of code saved.
There's a significant duplication of the field code:
Could be replaced by:
It's not clear how big the benefit is here. Also, maybe there's a better approach for this.
The text was updated successfully, but these errors were encountered: