Skip to content

Param alias#56

Open
DaymasS wants to merge 2 commits into
mainfrom
param-alias
Open

Param alias#56
DaymasS wants to merge 2 commits into
mainfrom
param-alias

Conversation

@DaymasS

@DaymasS DaymasS commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Изменения

Добавление алиасов для юзердаты.

Детали реализации

TODO.

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

@DaymasS DaymasS requested a review from petrCher June 25, 2026 17:43
@DaymasS DaymasS self-assigned this Jun 25, 2026
@github-actions

Copy link
Copy Markdown

💩 Code linting failed, use black and isort to fix it.

@github-actions

Copy link
Copy Markdown

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
userdata_api
   __main__.py17170%1–26
   exceptions.py19195%18
userdata_api/models
   base.py65592%25–28, 76
   db.py73297%97, 161
userdata_api/routes
   admin.py14379%25, 42–43
   category.py41490%38, 71–73
   exc_handlers.py20195%25
   param.py53885%39, 59, 98–101, 106, 129
   param_alias.py654038%19–29, 33–46, 50–54, 58–62, 76–80, 92–93, 101–104, 118–125, 139–141
   source.py30197%35
userdata_api/schemas
   base.py8450%6–9
   user.py24771%22–28
userdata_api/schemas/types
   scope.py24675%30–32, 37, 40, 42
userdata_api/utils
   admin.py261735%26–35, 52–98
   param_alias.py14193%49
   user.py86397%42, 84, 88
TOTAL70512083% 

Summary

Tests Skipped Failures Errors Time
57 0 💤 0 ❌ 0 🔥 5.133s ⏱️

@DaymasS

DaymasS commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

Убрать Category_id из поиска/инициализации алиаса

@petrCher

Copy link
Copy Markdown
Member

https://github.com/profcomff/dwh-pipelines/blob/main/dags/ods/user/user_to_back.py#L179
например здесь хочется только название параметра(алиаса) писать, без категории

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants