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

feat: settings, delete port validation #22

Merged
ivan.dev merged 3 commits from VORKOUT-31 into master 2025-12-02 11:41:58 +05:00
Conversation 3 Commits 3 Files Changed 32 +251 -479
ivan.dev commented 2025-11-15 16:00:20 +05:00
Member
Copy Link
No description provided.
ivan.dev added 1 commit 2025-11-15 16:00:20 +05:00
feat: settings, delete port validation 0330356ea2
ivan.dev requested review from cyrussmeat 2025-11-15 16:00:20 +05:00
ivan.dev requested review from mikhail.dev 2025-11-15 16:00:20 +05:00
ivan.dev requested review from vlad.dev 2025-11-15 16:00:20 +05:00
mikhail.dev reviewed 2025-11-17 13:44:41 +05:00
api/api/endpoints/ps_node.py Outdated
@@ -64,3 +64,1 @@
is_connected = await check_node_connection(
connection, ps_node_delete_data.node_id, ps_node_delete_data.next_node_id, int(ps_node_delete_data.port)
)
# is_connected = await check_node_connection(
mikhail.dev commented 2025-11-17 13:41:48 +05:00
Member
Copy Link

Если это ненужный код, то может лучше удалить?

Если это ненужный код, то может лучше удалить?
ivan.dev commented 2025-11-18 12:15:16 +05:00
Author
Member
Copy Link

Кирилл сказал на прошлом созвоне, пока проверку по портам закомментировать

Кирилл сказал на прошлом созвоне, пока проверку по портам закомментировать
cyrussmeat commented 2025-11-24 19:54:37 +05:00
Owner
Copy Link

С т.з. репозитория можно удалить в этой версии, мы всегда можем вернуться к этой ветке и позаимствовать код, согласен с Михаилом - не надо его тащить про запас

С т.з. репозитория можно удалить в этой версии, мы всегда можем вернуться к этой ветке и позаимствовать код, согласен с Михаилом - не надо его тащить про запас
👍 2
mikhail.dev marked this conversation as resolved
api/api/schemas/process/process_schema.py Outdated
@@ -38,3 +22,3 @@
class ProcessSchemaResponse(Base):
process_schema: ProcessSchema
node_listen: Ps_NodeFrontResponse
nodes: List[Ps_NodeFrontResponse]
mikhail.dev commented 2025-11-17 13:43:44 +05:00
Member
Copy Link

Вроде же на не старом питоне сидим, зачем тут deprecated тайпинги? Хотя, судя по всему, это не в одном месте проекта имеется.

Вроде же на не старом питоне сидим, зачем тут deprecated тайпинги? Хотя, судя по всему, это не в одном месте проекта имеется.
ivan.dev marked this conversation as resolved
ivan.dev added 1 commit 2025-11-18 12:13:00 +05:00
fix: deprecated typings a70973032f
ivan.dev added 1 commit 2025-11-24 20:02:36 +05:00
fix: delete port checks c86b7eb624
cyrussmeat approved these changes 2025-12-01 11:02:08 +05:00
vlad.dev approved these changes 2025-12-02 11:34:56 +05:00
ivan.dev merged commit fe7f8c6511 into master 2025-12-02 11:41:58 +05:00
Sign in to join this conversation.
Reviewers
No Reviewers
mikhail.dev
cyrussmeat
vlad.dev
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
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Vorkout/connect#22
Delete Branch "VORKOUT-31"

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?