Skip to content

Добавлена ручка delete для модалки - #31

Merged
petrCher merged 3 commits into
mainfrom
delete_modal
Jul 31, 2026
Merged

Добавлена ручка delete для модалки#31
petrCher merged 3 commits into
mainfrom
delete_modal

Conversation

@businkv

@businkv businkv commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Jul 28, 2026

Copy link
Copy Markdown

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
modal_backend
   __main__.py440%1–6
   exceptions.py20385%16, 32, 37
modal_backend/models
   base.py623150%22, 25–28, 52–60, 67–78, 83–88
   db.py881682%84–86, 90–93, 97–100, 104–108
modal_backend/routes
   exc_handlers.py17382%17, 31, 38
   groups.py25772%24–25, 33–34, 46, 62–63
   notes.py512159%70–73, 85–94, 110–113, 129–132, 148–151, 167–170, 186–189, 211–212, 224–226
   services.py25772%22–23, 36–37, 51–52, 64
modal_backend/schemas
   base.py12467%6–9
modal_backend/utils
   services.py905539%28–44, 48–86, 112–116, 120–122, 128–130, 140–144, 148–150, 156–158
TOTAL53615172% 

Summary

Tests Skipped Failures Errors Time
5 0 💤 0 ❌ 0 🔥 6.359s ⏱️

@businkv
businkv requested a review from petrCher July 28, 2026 21:27

@petrCher petrCher left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

исправь один недочет и замерджу

Comment thread modal_backend/routes/notes.py Outdated
@petrCher

Copy link
Copy Markdown
Member

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

@businkv

businkv commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

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

а ок

@petrCher
petrCher merged commit 1ef2f85 into main Jul 31, 2026
2 checks passed
@petrCher
petrCher deleted the delete_modal branch July 31, 2026 19:10
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