Implement login API, closes #1 #2

Open
BlackCorbeau wants to merge 25 commits from dev into main
Showing only changes of commit 2098719c82 - Show all commits
+22
View File
@@ -0,0 +1,22 @@
from dataclasses import dataclass
import json
from utils.token import generateKey
@dataclass
class user:
id: int
name: str
role: str
token: str
@classmethod
def initialize(cls, email:str, passwd:str):
Sweetbread marked this conversation as resolved
Review
  1. Отдели имя и аннотацию пробелами (email: str, ...)
  2. Зачем initialize (user.initialize(...)), когда уже есть __init__ (user(...))?
1. Отдели имя и аннотацию пробелами (`email: str`, ...) 2. Зачем `initialize` (`user.initialize(...)`), когда уже есть `__init__` (`user(...)`)?
#us = getUsModel() #возвращает словарь
id = 1#us['id']
name = 'Bob'#us['name']
role = 'Backend'#us['role']
Outdated
Review

Выглядит, как что-то недоработанное

Выглядит, как что-то недоработанное
Outdated
Review

Может потому что нет бд?)

Может потому что нет бд?)
Outdated
Review

Так а зачем PR отправил?

Так а зачем PR отправил?
token = generateKey(email, passwd)
return cls(id=id, name=name, role=role, token=token)
Outdated
Review

Попробуй просто по порядку перечислять аргументы, а то x=x так себе выглядит

Попробуй просто по порядку перечислять аргументы, а то `x=x` так себе выглядит
def toJSON(self):
return json.dumps({"token": f'{self.token}', "user": {"id": self.id, "role": f'{self.role}', "name": f'{self.name}'}})
Outdated
Review
  1. Зачем f-строки? Если переменные и так строки, то просто вставляй их
  2. Стоит ли преобразовывать JSON в строку?
1. Зачем f-строки? Если переменные и так строки, то просто вставляй их 2. Стоит ли преобразовывать JSON в строку?