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

feat: added endpoints: auth, pofile, account, keyring #5

Merged
ivan.dev merged 10 commits from VORKOUT-4 into master 2025-05-15 15:21:33 +05:00
Conversation 0 Commits 10 Files Changed 29 +3 -2
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit b90b70568c - Show all commits

3
api/api/endpoints/account.py
View File

@ -67,7 +67,6 @@ async def create_account(
if user_validation is None:
await create_user(connection,user,authorize_user.id)
user_new = await get_user_login(connection, user.login)
return user_new
@ -104,7 +103,7 @@ async def update_account(
if update_values is None:
return user
user_update_data = User.model_validate(update_values)
user_update_data = User.model_validate({**user.model_dump(), **update_values})
ivan.dev marked this conversation as resolved
vlad.dev commented 2025-04-24 12:35:43 +05:00
Review
Copy Link

А вот где неиспользуемые переменные, можешь объяснить зачем они, я просто не особо понял?
Вот к примеру user_update_data и authorize_user, это какой-то задел на будущее?

А вот где неиспользуемые переменные, можешь объяснить зачем они, я просто не особо понял? Вот к примеру `user_update_data` и `authorize_user`, это какой-то задел на будущее?
ivan.dev commented 2025-04-29 21:17:59 +05:00
Review
Copy Link

Да, если логика поменяется.

Да, если логика поменяется.
await update_user_id(connection, update_values, user)

2
api/api/endpoints/keyring.py
View File

@ -107,6 +107,8 @@ async def update_keyring(
if update_values is None:
return keyring
keyring_update_data = AccountKeyring.model_validate({**keyring.model_dump(), **update_values})
await update_key_id(connection, update_values, keyring)