feat: CRUD process schema #19

Merged
ivan.dev merged 2 commits from VORKOUT-21 into master 2025-10-17 12:28:53 +05:00
Member
No description provided.
ivan.dev added 1 commit 2025-10-13 11:51:16 +05:00
ivan.dev changed title from feat: CRUD process schema to WIP: feat: CRUD process schema 2025-10-13 11:51:18 +05:00
ivan.dev requested review from cyrussmeat 2025-10-13 11:51:26 +05:00
ivan.dev requested review from mikhail.dev 2025-10-13 11:51:26 +05:00
ivan.dev requested review from vlad.dev 2025-10-13 11:51:27 +05:00
cyrussmeat requested changes 2025-10-15 17:17:19 +05:00
Dismissed
@@ -0,0 +42,4 @@
return Ps_Node.model_validate(ps_node_data)
async def create_ps_node_start_schema(
Owner

Я поглядел, что реализация в process_schema не задействует данный метод. Он нужен?

Я поглядел, что реализация в process_schema не задействует данный метод. Он нужен?
Author
Member

Это когда стартовая нода была, удалю.

Это когда стартовая нода была, удалю.
ivan.dev marked this conversation as resolved
@@ -0,0 +64,4 @@
# return validated_listen_schema
async def get_last_ps_node_by_creator_and_ps_id(
Owner

Это ты эмулируешь last_insert_id = await database.execute(query) ?

Это ты эмулируешь last_insert_id = await database.execute(query) ?
ivan.dev marked this conversation as resolved
@@ -113,0 +131,4 @@
await create_process_schema(connection, user_validation.id, title, description)
process_schema_new = await get_process_schema_by_title(connection, title)
Owner

А почему в такой связке не используется last_insert_id и получение схемы по идентификатору из БД, который PK?

А почему в такой связке не используется last_insert_id и получение схемы по идентификатору из БД, который PK?
Author
Member

Логично, сейчас переделаю.

Логично, сейчас переделаю.
ivan.dev marked this conversation as resolved
ivan.dev added 1 commit 2025-10-15 17:50:43 +05:00
cyrussmeat approved these changes 2025-10-15 19:20:05 +05:00
vlad.dev approved these changes 2025-10-16 12:04:43 +05:00
ivan.dev changed title from WIP: feat: CRUD process schema to feat: CRUD process schema 2025-10-17 11:56:55 +05:00
ivan.dev merged commit e4a9fe6d01 into master 2025-10-17 12:28:53 +05:00
Sign in to join this conversation.
No Label
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Vorkout/connect#19
No description provided.