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, AllProcessSchemaResponse
async 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 None
processschema_data = {
Тут можно упростить, написал ниже
@@ -0,0 +142,4 @@
if not processschema_db:
return None
processschema_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] = None
order: Optional[Dict[str, str]] = None # {field: "id", direction: "asc"}
filters: Optional[Dict[str, List[str]]] = None # {"status": ["active"], "owner_id": ["123"]}
Чтобы не писать такие комментарии, можешь использовать
TypedDict
.