Vorkout/connect
4
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: endpoints services #21

Merged
ivan.dev merged 1 commits from codestyle-issue-refactor into master 2025-11-11 18:05:30 +05:00
Conversation 5 Commits 1 Files Changed 17 +727 -359
ivan.dev commented 2025-11-06 23:35:03 +05:00
Member
Copy Link
No description provided.
ivan.dev added 1 commit 2025-11-06 23:35:04 +05:00
feat: endpoints services 3933a05582
ivan.dev requested review from cyrussmeat 2025-11-06 23:35:09 +05:00
ivan.dev requested review from mikhail.dev 2025-11-06 23:35:09 +05:00
ivan.dev requested review from vlad.dev 2025-11-06 23:35:09 +05:00
cyrussmeat commented 2025-11-07 12:31:58 +05:00
Owner
Copy Link

Стало сильно лучше.
Вопрос, по организации папок только services/endpoints типа ты руководствовался тем, что это сервисные классы только "под эндпоинты" ?

Стало сильно лучше. Вопрос, по организации папок только services/endpoints типа ты руководствовался тем, что это сервисные классы только "под эндпоинты" ?
❤️ 1
vlad.dev approved these changes 2025-11-07 13:45:16 +05:00
vlad.dev left a comment
Member
Copy Link

Единственное, что увидел
Можно вынести создание сервисов в фабрику, чтобы делать это через механизм fastapi
Но по сути разницы особой не будет, потому что в зависимостях только connection и ничего больше не прибавится

Единственное, что увидел Можно вынести создание сервисов в фабрику, чтобы делать это через механизм `fastapi` Но по сути разницы особой не будет, потому что в зависимостях только `connection` и ничего больше не прибавится
❤️ 1
ivan.dev commented 2025-11-07 15:05:54 +05:00
Author
Member
Copy Link

Стало сильно лучше.
Вопрос, по организации папок только services/endpoints типа ты руководствовался тем, что это сервисные классы только "под эндпоинты" ?

Да, Под каждый ендпоинт, свой класс.

> Стало сильно лучше. > Вопрос, по организации папок только services/endpoints типа ты руководствовался тем, что это сервисные классы только "под эндпоинты" ? Да, Под каждый ендпоинт, свой класс.
cyrussmeat commented 2025-11-07 17:02:47 +05:00
Owner
Copy Link

Да, Под каждый ендпоинт, свой класс.

Окей, мне просто показалось, что сервисы могут иметь более широкий прикладной кейс и для других точек входа (например для консольного управления). В целом оно не суть важно, чисто семантика путей.

> Да, Под каждый ендпоинт, свой класс. Окей, мне просто показалось, что сервисы могут иметь более широкий прикладной кейс и для других точек входа (например для консольного управления). В целом оно не суть важно, чисто семантика путей.
❤️ 1
cyrussmeat approved these changes 2025-11-07 17:18:57 +05:00
ivan.dev merged commit 46091380ba into master 2025-11-11 18:05:30 +05:00
Sign in to join this conversation.
Reviewers
No Reviewers
mikhail.dev
vlad.dev
cyrussmeat
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
cyrussmeat ivan.dev mikhail.dev vlad.dev
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Vorkout/connect#21
Delete Branch "codestyle-issue-refactor"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?