Vorkout/connect
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

VORKOUT-8 #13

Merged
vlad.dev merged 30 commits from VORKOUT-8 into master 2025-07-02 12:23:44 +05:00
Conversation 0 Commits 30 Files Changed 23 +4 -4
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ad1369c3e3 - Show all commits

8
api/api/schemas/endpoints/account.py
View File

@ -22,14 +22,14 @@ class UserUpdate(Base):
class UserCreate(Base): class UserCreate(Base):
cyrussmeat commented 2025-06-30 16:23:40 +05:00
Review
Copy Link

Валидация идёт на уровне этой модели или на другом дескрипторе?

Просто тут name, login, role, status не должны быть Optional.

Валидация идёт на уровне этой модели или на другом дескрипторе? Просто тут name, login, role, status не должны быть Optional.
vlad.dev commented 2025-06-30 16:28:34 +05:00
Review
Copy Link

На уровне этой, поправлю на обязательные

На уровне этой, поправлю на обязательные
name: Optional[str] = Field(None, max_length=100) name: str = Field(max_length=100)
login: Optional[str] = Field(None, max_length=100) login: str = Field(max_length=100)
email: Optional[EmailStr] = None email: Optional[EmailStr] = None
password: Optional[str] = None password: Optional[str] = None
bind_tenant_id: Optional[str] = Field(None, max_length=40) bind_tenant_id: Optional[str] = Field(None, max_length=40)
role: Optional[AccountRole] = None role: AccountRole
meta: Optional[dict] = None meta: Optional[dict] = None
status: Optional[AccountStatus] = None status: AccountStatus
class AllUser(Base): class AllUser(Base):