Skip to content

Исправление логики работы patch#49

Merged
Wudext merged 3 commits intomainfrom
Patch-order-duplication-fix
Apr 6, 2024
Merged

Исправление логики работы patch#49
Wudext merged 3 commits intomainfrom
Patch-order-duplication-fix

Conversation

@Wudext
Copy link
Copy Markdown
Contributor

@Wudext Wudext commented Apr 5, 2024

Изменения

Изменил логику работы фильтрации, при выполнении patch запроса для кнопок.

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

Добавил проверку по id категории, которой раньше не было. Из-за этого происходили дубли по полю order

Check-List

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 5, 2024

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
services_backend
   __main__.py440%1–7
   schemas.py8450%6–9
services_backend/models
   base.py13469%19–22
services_backend/routes
   button.py1151983%66, 88–93, 110, 115, 132, 137, 167, 173, 208–213
   category.py109595%90, 118, 150–151, 192
TOTAL3323689% 

Summary

Tests Skipped Failures Errors Time
29 0 💤 0 ❌ 0 🔥 1.487s ⏱️

@dyakovri dyakovri force-pushed the Patch-order-duplication-fix branch from 0eb942c to 6d3ff22 Compare April 6, 2024 12:17
@Wudext Wudext merged commit f063932 into main Apr 6, 2024
@Wudext Wudext deleted the Patch-order-duplication-fix branch April 6, 2024 12:52
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