Skip to content

38 create documentation#97

Open
Dary5 wants to merge 1 commit intomainfrom
38-create-documentation
Open

38 create documentation#97
Dary5 wants to merge 1 commit intomainfrom
38-create-documentation

Conversation

@Dary5
Copy link
Contributor

@Dary5 Dary5 commented Jun 5, 2023

Task merge request

Related Issue

#38

Reviewer requrements

Особых требований нет

Code changes

Описание классов, сервисов и репозиториев.

@Dary5 Dary5 force-pushed the 38-create-documentation branch from a55e6b3 to 541b89d Compare June 5, 2023 17:23
Comment on lines +72 to +77
<details>
<summary>
<code> Calendar.py </code>
</summary>

<br> Это сущность не будет использована в MVP.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

это я говорила выпилить

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

готово

Comment on lines +101 to +108
<details>
<summary>
<code> User.py </code>
</summary>

| имя | тип | описание |
|:---:|:---:|:-------------------------------------:|
| id |int| уникальный идентификатор пользователя |
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

выпилить

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

готово

Comment on lines +191 to +135
├── Services
├── LocalTaskService
├── CalDavService
├── ServerService
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

неверные названия серверов

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут только LocalTaskService -> TaskService

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

я знаю, если исправила - отлично


```

### Описание TaskService
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

лишнее, убрать

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

пробелы в смысле?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

убрать ### TaskService

Comment on lines +204 to +152
TaskService - скрывает в себе работу с локальной базой данных, а также валидацией данных.
<details>
<summary>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

этот кусок выглядит некрасиво, вставляй тэг: <br> для перехода на следующую строку

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

сделано

CalDavService - скрывает в себе работу с удаленным сервером по протоколу CalDAV.
<details>
<summary>
<code> Метод get_all_tasks </code>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

и вдруг родилось слово "Метод" в тэгах code...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

исправлено

Comment on lines +399 to +401
| Выходные параметры | Тип |
|:-------------------------------------------------:|:----:|
| Task entity - <br/> self.__task_from_event(event) | Task |
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

если я, как программист, который хочет воспользоваться этой документацией, открою и увижу этот тип - у меня глаза вылезут из орбит от удивления, что здесь происходит и какой наркоман это писал

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

поправлю на просто Task без названия возвращаемого параметра

<code> ServerService.py </code>
</summary>

ServerService - скрывает в себе работу с серверами, на которых хранятся задачи.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

этот сервис, который работает с серверами в БД, а не с каким-то серверами, которые хранят задачи

<summary>
<code> get_count </code>
</summary>
<br> Метод возвращает количество заданных email в репозитории.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше звучит:
Возвращает число записей, соответствующих переданному email

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

исправлено

Comment on lines +522 to +524
</summary>
<br> Метод возвращает список статусов по id сервера, на котором они хранятся.
Если был передан не id, то возникает ошибка Invalid("Невозможно открыть список статусов").
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Возвращает список статусов, которые соответствуют определенному серверу

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

исправлено

@Dary5 Dary5 force-pushed the 38-create-documentation branch from 8e35736 to 6ea37fd Compare June 6, 2023 01:03
@Dary5 Dary5 force-pushed the 38-create-documentation branch from 23268a8 to 3da4a14 Compare June 6, 2023 18:47
Comment on lines +55 to +68

| имя | тип | описание |
|:---------------:|:---:|:--------:|
| id |int|уникальный идентификатор задачи|
| number |int|номер задачи на текущем уровне|
| path |text|путь до этой задачи|
| dtstamp |date| дата создания задачи|
| dtstart |date| дата начала задачи|
| due |date| дата окончания задачи|
| category |list| категория задачи|
| summary |text| наименование задачи|
| status |int| статус задачи (выполнена/не выполнена)|
|technical_status |int| технический статус задачи (создана, удалена, изменена...)|
| description |text| описание задачи|
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

сущность нужно обновить, у нее старые поля

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
обновила

@Dary5 Dary5 force-pushed the 38-create-documentation branch 3 times, most recently from 32e1a7f to 9527846 Compare June 8, 2023 00:30
@Dary5
Copy link
Contributor Author

Dary5 commented Jun 8, 2023

@astronik00 добавила описание объекта Session и описала основные методы, проверь пожалуйста

@Dary5 Dary5 force-pushed the 38-create-documentation branch 2 times, most recently from 6f73964 to 2e922e6 Compare June 8, 2023 16:02
@Dary5 Dary5 force-pushed the 38-create-documentation branch from 457337e to 71a9bcf Compare June 8, 2023 21:18
@Dary5 Dary5 force-pushed the 38-create-documentation branch from 71a9bcf to f23be7d Compare June 8, 2023 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants