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

feat: settings, delete port validation #22

Open
ivan.dev wants to merge 3 commits from VORKOUT-31 into master
pull from: VORKOUT-31
merge into: Vorkout:master
Conversation 1 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
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin VORKOUT-31:VORKOUT-31
git checkout VORKOUT-31
Sign in to join this conversation.
Reviewers
No Reviewers
vlad.dev
cyrussmeat
mikhail.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
3 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?