feat: CRUD ProcessSchema #16
		Reference in New Issue
	
	Block a user
	
	No description provided.
		
		Delete Branch "VORKOUT-17"
	
	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?
@@ -0,0 +13,4 @@from api.schemas.endpoints.process_schema import all_process_schema_adapter, AllProcessSchemaResponseasync def get_processschema_page_by_creator_id(Может, переименуем
processschemaвprocess_schema, а то как-то 3 s странновато выглядят?@@ -0,0 +64,4 @@async def get_processschema_page(connection: AsyncConnection, page, limit) -> Optional[AllProcessSchemaResponse]:"""Получает список событий заданного создателя по значениям page и limit.Тут, наверное, тогда не "заданного", если я правильно понял
@@ -0,0 +119,4 @@if not processschema_db:return Noneprocessschema_data = {Тут можно упростить, написал ниже
@@ -0,0 +142,4 @@if not processschema_db:return Noneprocessschema_data = {Давай тут перейдем к схеме попроще
@@ -57,3 +57,1 @@list_eventslist = await get_listevents_page(connection, page, limit)print(list_eventslist)if list_eventslist is None:list_eventspage = await get_listevents_page(connection, page, limit)list_events_page?@@ -155,3 +154,4 @@authorize_user = await db_user_role_validation_for_listevents_and_processschema_by_listevent_id(connection, current_user, listevents_validation.creator_id)Давай получение полей для обновления тоже упростим
Посмотри еще где можно, вроде пару мест точно видел)
by_alias сделал потому, что при имени schema была ошибка совпадения имен (schema_)
В текущей вариации обновления полей не работает как раз таки со схемой
@@ -0,0 +38,4 @@api_router = APIRouter(prefix="/processschema",Может все-таки везде будем делить
process-schemaна 2 слова? Просто уж очень странно это выглядит одинЦепочками с дефисом или нижним подчёркиванием удобнее читать и меньше шансов словить ошибку восприятия, ага
Ну, кроме наименований с моей стороны все ок, осталось только дождаться мнения @cyrussmeat
И перед мержем не забудь отформатировать код
@@ -0,0 +61,4 @@)async def get_process_schema_page(connection: AsyncConnection, page, limit) -> Optional[AllProcessSchemaResponse]:А если какая-то дополнительная фильтрация будет над этим списком - отдельным методом будет реализовываться?
Это я к тому, что я бы через DTO расширяемый объект фильтрации тут проводил, у которого по базе limit[offset,count], search и order, к примеру
@@ -0,0 +159,4 @@@api_router.delete("/{processschema_id}", dependencies=[Depends(bearer_schema)], response_model=ProcessSchema)async def delete_process_schema(processschema_id: int,Получится привести все использования этого слова во всех файлах к одному виду? О чём писали в комментариях выше.
@@ -0,0 +42,4 @@pagination: Dict[str, int] # {page: 1, limit: 10}search: Optional[str] = Noneorder: Optional[Dict[str, str]] = None # {field: "id", direction: "asc"}filters: Optional[Dict[str, List[str]]] = None # {"status": ["active"], "owner_id": ["123"]}Чтобы не писать такие комментарии, можешь использовать
TypedDict.