Param alias - #56
Conversation
|
💩 Code linting failed, use |
Coverage Report
Summary
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Убрать Category_id из поиска/инициализации алиаса |
|
https://github.com/profcomff/dwh-pipelines/blob/main/dags/ods/user/user_to_back.py#L179 |
|
💩 Code linting failed, use |
1 similar comment
|
💩 Code linting failed, use |
|
@DaymasS сделай ребейз (исправил линтинг) |
|
💩 Code linting failed, use |
|
💩 Code linting failed, use |
|
💩 Code linting failed, use |
| param_alias = APIRouter(prefix="/param/{param_id}/alias", tags=["Param Alias"]) | ||
|
|
||
|
|
||
| def _get_param(*, param_id: int) -> Param: |
There was a problem hiding this comment.
а зачем такую функцию делать если есть кастомный метод Param.get который вернет 404? я бы даже без функции просто в коде бы прописал в самой ручке
There was a problem hiding this comment.
но если хочешь оставить функцию по каким-то причинам, то я не уверен, что надо что-то возвращать, ты же потом не используешь полученные данные
| def _validate_source(source_id: int | None) -> None: | ||
| if source_id is None: | ||
| return | ||
| source = Source.query(session=db.session).filter(Source.id == source_id).one_or_none() |
There was a problem hiding this comment.
здесь тоже можно было бы заменить на Source.get
|
|
||
| @param_alias.post("", response_model=ParamAliasGet) | ||
| async def create_param_alias( | ||
| request: Request, |
| request: Request, | ||
| param_id: int, | ||
| alias_inp: ParamAliasPost, | ||
| _: dict[str, Any] = Depends(UnionAuth(scopes=["userdata.param.create"], allow_none=False, auto_error=True)), |
There was a problem hiding this comment.
есть идея скоупы сделать для алиасов конкретно
по типу userdata.alias.create
просто я бы наверное мог выдавать доступ команде именно к этим доступам, чтобы сами параметры не могли поменять, но могли работать с алиасами
|
|
||
| @param_alias.delete("/{alias_id}", response_model=StatusResponseModel) | ||
| async def delete_param_alias( | ||
| request: Request, |
|
запусти сейчас make format, у меня локально для текущего кода проблемы, на которые ругается линтинг исчезают |
| Может быть привязан к конкретному источнику или быть общим для всех источников. | ||
| """ | ||
|
|
||
| name: Mapped[str] = mapped_column(String, unique=True) |
There was a problem hiding this comment.
unique=True заблочит создание алиаса с названием удаленного (у нас же софт делиты)
|
|
||
|
|
||
| class ParamAliasPatch(Base): | ||
| name: constr(min_length=1) | None = None |
There was a problem hiding this comment.
здесь возможно none передать, поэтому в userdata_api/routes/param_alias.py в строке patch_data = alias_inp.model_dump(exclude_unset=True) будет передано поле с none
дальше _check_alias_name_exists не вылетит, и код дойдет до alias = ParamAlias.update
но name не может быть none
2 participants
Изменения
Добавление алиасов для юзердаты.
Детали реализации
TODO.
Check-List
blackиisortдля Back-End илиPrettierдля Front-End?