Skip to content

Commit

Permalink
[INTERNAL] enforce required email field
Browse files Browse the repository at this point in the history
## Summary of changes

The proto for `user` was created before our annotation pattern was finalized. As a result it is too permissive. It make the `email` field optional, but it is actually required.

## Testing strategy

Borrow and tested in console-eng

GitOrigin-RevId: 6a23ceb82b50fd37691b10c8dd0e8f7e51eeb0e9
  • Loading branch information
aray12 committed Oct 17, 2024
1 parent f2edf2b commit 3663930
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/gretel_client/rest_v1/models/user_profile.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ class UserProfile(BaseModel):
firstname: Optional[StrictStr] = None
lastname: Optional[StrictStr] = None
image: Optional[UserProfileImage] = None
email: Optional[StrictStr] = None
email: StrictStr
__properties: ClassVar[List[str]] = [
"id",
"firstname",
Expand Down

0 comments on commit 3663930

Please sign in to comment.