Feat 8.frontend development #22

Merged
LazIvanS merged 7 commits from feat_8.frontend_development into develop 2021-04-26 15:37:09 +03:00
LazIvanS commented 2021-04-22 21:28:41 +03:00 (Migrated from github.com)
No description provided.
AlekseyLobanov (Migrated from github.com) reviewed 2021-04-26 13:07:20 +03:00
@@ -0,0 +9,4 @@
def __init__(self, master=None, url=None) -> None:
"""
Функция инициаизации класса
AlekseyLobanov (Migrated from github.com) commented 2021-04-23 00:16:45 +03:00

Такой комментарий очень похож на очевидный. Очевидные комментари лучше не вставлять, т.к. они только затрудняют чтение

Такой комментарий очень похож на очевидный. Очевидные комментари лучше не вставлять, т.к. они только затрудняют чтение
@@ -9,0 +25,4 @@
def login(self):
"""Возвращает пользователя - его можно потом сериализовать"""
self.frame = LoginFrame(master=self, url=DEFAULT_URL)
while not self.frame.loggedIn:
AlekseyLobanov (Migrated from github.com) commented 2021-04-26 13:06:20 +03:00

Возможно, полезно было бы сохранять JWT токен в отдельный файл, чтобы логиниться только когда токен протухнет

Возможно, полезно было бы сохранять JWT токен в отдельный файл, чтобы логиниться только когда токен протухнет
@@ -0,0 +100,4 @@
def removeUserList(self, id):
self.items = [item for item in self.items if item.id != id]
# ToDo
AlekseyLobanov (Migrated from github.com) commented 2021-04-26 13:07:03 +03:00

Непонятно, что делает ToDo. Если это маркер для того, что надо сделать, то лучше писать, что именно сделать

Непонятно, что делает ToDo. Если это маркер для того, что надо сделать, то лучше писать, что именно сделать
LazIvanS (Migrated from github.com) reviewed 2021-04-26 13:10:27 +03:00
@@ -0,0 +100,4 @@
def removeUserList(self, id):
self.items = [item for item in self.items if item.id != id]
# ToDo
LazIvanS (Migrated from github.com) commented 2021-04-26 13:10:27 +03:00

Реализовать ¯\(ツ)

Реализовать ¯\\_(ツ)_/¯
LazIvanS (Migrated from github.com) reviewed 2021-04-26 13:10:50 +03:00
@@ -9,0 +25,4 @@
def login(self):
"""Возвращает пользователя - его можно потом сериализовать"""
self.frame = LoginFrame(master=self, url=DEFAULT_URL)
while not self.frame.loggedIn:
LazIvanS (Migrated from github.com) commented 2021-04-26 13:10:50 +03:00

Я там в одном из комментов предлагал сделать юзера json-сериализуемым

Я там в одном из комментов предлагал сделать юзера json-сериализуемым
LazIvanS (Migrated from github.com) reviewed 2021-04-26 13:11:18 +03:00
@@ -0,0 +9,4 @@
def __init__(self, master=None, url=None) -> None:
"""
Функция инициаизации класса
LazIvanS (Migrated from github.com) commented 2021-04-26 13:11:17 +03:00

Осталось от кода Александра

Осталось от кода Александра
AlekseyLobanov (Migrated from github.com) reviewed 2021-04-26 13:12:16 +03:00
@@ -9,0 +25,4 @@
def login(self):
"""Возвращает пользователя - его можно потом сериализовать"""
self.frame = LoginFrame(master=self, url=DEFAULT_URL)
while not self.frame.loggedIn:
AlekseyLobanov (Migrated from github.com) commented 2021-04-26 13:12:15 +03:00

Зачем его сериализовать? Достаточно сохранить просто токен и добавить загрузку этого токена при запуске.
Логин\пароль и другие данные не нужны, их можно подгружать.

Зачем его сериализовать? Достаточно сохранить просто токен и добавить загрузку этого токена при запуске. Логин\пароль и другие данные не нужны, их можно подгружать.
LazIvanS (Migrated from github.com) reviewed 2021-04-26 13:48:00 +03:00
@@ -9,0 +25,4 @@
def login(self):
"""Возвращает пользователя - его можно потом сериализовать"""
self.frame = LoginFrame(master=self, url=DEFAULT_URL)
while not self.frame.loggedIn:
LazIvanS (Migrated from github.com) commented 2021-04-26 13:47:59 +03:00

Ну пл сути все что будет в сериализации это пара токенов и GUI будет известен только юзер

Ну пл сути все что будет в сериализации это пара токенов и GUI будет известен только юзер
Sign in to join this conversation.