Skip to content

added validation when create - #41

Open
petrCher wants to merge 2 commits into
mainfrom
validation_when_create
Open

petrCher wants to merge 2 commits into
mainfrom
validation_when_create

Conversation

@petrCher

@petrCher petrCher commented Aug 27, 2026

Copy link
Copy Markdown
Member

Изменения

Валидация модалок при создании в зависимости от типа

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

Check-List

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

@petrCher petrCher self-assigned this Aug 27, 2026
@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
modal_backend
   __main__.py440%1–6
modal_backend/models
   base.py62789%22, 25–28, 57, 87
modal_backend/routes
   notes.py66198%251
modal_backend/schemas
   base.py12467%6–9
modal_backend/utils
   user_logic.py22291%22, 26
TOTAL6101897% 

Summary

Tests Skipped Failures Errors Time
110 0 💤 0 ❌ 0 🔥 8.006s ⏱️

@Aiz0r

Aiz0r commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Вроде все поправил, жду ревью @petrCher.

@Aiz0r

Aiz0r commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

ну или от @Georgon тоже жду ревью

@petrCher

Copy link
Copy Markdown
Member Author

git rebase сделай, так как залил твой прошлый пр
ревью этого скоро будет

@petrCher

Copy link
Copy Markdown
Member Author

если что только надо меня и Гошу не в assigners добавлять, а в reviews)
ты же ревью просишь
а на assigners себя

@Aiz0r

Aiz0r commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

если что только надо меня и Гошу не в assigners добавлять, а в reviews)
ты же ревью просишь
а на assigners себя

Ой, сорри, поправлю завтра

@Aiz0r

Aiz0r commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

git rebase сделай, так как залил твой прошлый пр
ревью этого скоро будет

Ок, поправлю

@Aiz0r
Aiz0r force-pushed the validation_when_create branch from bce5048 to 17ef578 Compare September 20, 2026 17:51
@Aiz0r

Aiz0r commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

@petrCher @Georgon rebase сделал, жду ревью

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.

добавить валидацию полей при создании note

3 participants