alex/django-todo

Implemented most of the api placeholders #23

Merged

LazIvanS merged 7 commits from feat_9.functional_api into develop

2021-04-28 23:02:43 +03:00

LazIvanS commented

2021-04-26 22:46:01 +03:00

(Migrated from github.com)

No description provided.

AlekseyLobanov (Migrated from github.com) reviewed 2021-04-26 23:57:12 +03:00

AlekseyLobanov (Migrated from github.com) left a comment

В целом ок, но надо удалить numpy.
Как мне, кажется, можно немного причесать код, чтобы удалить какие-то лишние комментарии, использовать logging вместо принтов, лишние убрать.

В целом ок, но надо удалить numpy. Как мне, кажется, можно немного причесать код, чтобы удалить какие-то лишние комментарии, использовать logging вместо принтов, лишние убрать.

frontend/api_demo.py

AlekseyLobanov (Migrated from github.com) commented

2021-04-26 23:53:26 +03:00

Зачем тут numpy? Лучше нинада

Зачем тут numpy? Лучше нинада

frontend/api_demo.py

						
				@@ -8,4 +11,4 @@

				DEFAULT_URL = "http://127.0.0.1:8000"

				user = User(url=DEFAULT_URL)

AlekseyLobanov (Migrated from github.com) commented

2021-04-26 23:53:55 +03:00

user.lists_[0].modify(title=f"A new title {random.random()}")

```suggestion user.lists_[0].modify(title=f"A new title {random.random()}") ```

frontend/user.py

AlekseyLobanov (Migrated from github.com) commented

2021-04-26 23:54:31 +03:00

            created_item = ToDoItem(id=random.randint(100, 1000), text=text, user=self.user)

```suggestion created_item = ToDoItem(id=random.randint(100, 1000), text=text, user=self.user) ```

AlekseyLobanov (Migrated from github.com) commented

2021-04-26 23:55:12 +03:00

Кажется, лишний print

Кажется, лишний print

frontend/user.py

						
				@@ -83,3 +137,4 @@

				    return [

				        ToDoList(

				            id=i,

				            title=f"List {i}",

AlekseyLobanov (Migrated from github.com) commented

2021-04-26 23:55:48 +03:00

Не python_style_name. Лучше to_do_lists

Не python_style_name. Лучше `to_do_lists`

AlekseyLobanov (Migrated from github.com) commented

2021-04-26 23:56:10 +03:00

По идее, у каждого элемента должен быть родитель

По идее, у каждого элемента должен быть родитель

LazIvanS commented

2021-04-27 17:07:43 +03:00

(Migrated from github.com)

Поправил все из ревью. Про логгинг не знаю, нужен ли он нам, когда времени особо нет, в принципе всегда можно stdout перенаправлять при желании если вывод причесать.

Поправил все из ревью. Про логгинг не знаю, нужен ли он нам, когда времени особо нет, в принципе всегда можно stdout перенаправлять при желании если вывод причесать.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: alex/django-todo#23