Implement login API, closes #1 #2

Open
BlackCorbeau wants to merge 25 commits from dev into main
Showing only changes of commit 5058df8b7d - Show all commits
+7 -9
View File
@@ -9,14 +9,12 @@ class user:
role: str
token: str
@classmethod
def initialize(cls, email:str, passwd:str):
def __init__(self, email: str, passwd: str):
#us = getUsModel() #возвращает словарь
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(...)`)?
id = 1#us['id']
name = 'Bob'#us['name']
role = 'Backend'#us['role']
token = generateKey(email, passwd)
return cls(id=id, name=name, role=role, token=token)
self.id = 1#us['id']
self.name = 'Bob'#us['name']
self.role = 'Backend'#us['role']
self.token = generateKey(email, passwd)
Outdated
Review

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

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

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

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

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

Так а зачем PR отправил?
def toJSON(self):
return json.dumps({"token": f'{self.token}', "user": {"id": self.id, "role": f'{self.role}', "name": f'{self.name}'}})
def toDictionary(self):
Outdated
Review

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

Попробуй просто по порядку перечислять аргументы, а то `x=x` так себе выглядит
return {"user": {"id": self.id, "name": self.name, "role": self.role}, "token": self.token}