-
-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Проблема с добавлением полей для сортировки в запрос #253
Conversation
Руководство для рецензента от SourceryPull-запрос решает проблему с добавлением полей для сортировки в запросе. Реализация добавляет новые методы в список исключений, предотвращая добавление параметра сортировки к этим методам. Диаграмма класса, показывающая обновленный класс UserRequestclassDiagram
class UserRequest {
-EXCLUDED_METHODS: Set[str]
+add_order_parameter()
}
note for UserRequest "Добавлено 3 новых метода в EXCLUDED_METHODS:
- voximplant.statistic.get
- crm.deal.userfield.list
- task.elapseditem.getlist"
Изменения на уровне файлов
Оценка по связанным задачам
Советы и командыВзаимодействие с Sourcery
Настройка вашего опытаПолучите доступ к вашей панели управления, чтобы:
Получение помощи
Original review guide in EnglishReviewer's Guide by SourceryThe pull request addresses a problem with adding fields for sorting in a request. The implementation adds new methods to the exclusion list, preventing the addition of the order parameter to these methods. Class diagram showing the updated UserRequest classclassDiagram
class UserRequest {
-EXCLUDED_METHODS: Set[str]
+add_order_parameter()
}
note for UserRequest "Added 3 new methods to EXCLUDED_METHODS:
- voximplant.statistic.get
- crm.deal.userfield.list
- task.elapseditem.getlist"
File-Level Changes
Assessment against linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Привет, @leshchenko1979 - я просмотрел ваши изменения, и они выглядят отлично!
Вот что я рассмотрел во время проверки
- 🟢 Общие вопросы: все выглядит хорошо
- 🟢 Безопасность: все выглядит хорошо
- 🟢 Тестирование: все выглядит хорошо
- 🟢 Сложность: все выглядит хорошо
- 🟢 Документация: все выглядит хорошо
Sourcery бесплатен для open source - если вам нравятся наши обзоры, пожалуйста, подумайте о том, чтобы поделиться ими ✨
Original comment in English
Hey @leshchenko1979 - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Спасибо за быстрый фикс. |
Fixes #98
Сводка от Sourcery
Исправления ошибок:
voximplant.statistic.get
,crm.deal.userfield.list
иtask.elapseditem.getlist
.Original summary in English
Summary by Sourcery
Bug Fixes:
voximplant.statistic.get
,crm.deal.userfield.list
, andtask.elapseditem.getlist
.