Skip to content

Commit a9eca87

Browse files
authored
Merge pull request #99 from PrivateAIM/bump-deps
Bump dependencies and fix tests for creating users
2 parents edb85d2 + 541a871 commit a9eca87

4 files changed

Lines changed: 372 additions & 322 deletions

File tree

flame_hub/_auth_client.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
import typing as t
44

55
import typing_extensions as te
6-
from pydantic import BaseModel, Field, WrapValidator
6+
from pydantic import BaseModel, Field, WrapValidator, EmailStr
77

88
from flame_hub._base_client import (
99
BaseClient,
@@ -55,7 +55,7 @@ class User(BaseModel):
5555
name: str
5656
active: bool
5757
name_locked: bool
58-
email: t.Annotated[str | None, IsOptionalField] = None
58+
email: t.Annotated[EmailStr, IsOptionalField] = None
5959
display_name: str | None
6060
first_name: str | None
6161
last_name: str | None
@@ -458,8 +458,8 @@ def find_role_permissions(self, **params: te.Unpack[FindAllKwargs]) -> list[Role
458458
def create_user(
459459
self,
460460
name: str,
461+
email: str,
461462
display_name: str = None,
462-
email: str = None,
463463
active: bool = True,
464464
name_locked: bool = True,
465465
first_name: str = None,

0 commit comments

Comments
 (0)