Skip to content
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

Feature/recipe additional fields #216

Merged
merged 10 commits into from
Dec 25, 2023
Merged

Conversation

earlinn
Copy link
Collaborator

@earlinn earlinn commented Dec 25, 2023

Добавила в сериализатор рецептов поля amount (количества продукта в упаковке), final_price (цена продукта с учетом скидки), quantity_in_recipe (раньше оно называлось quantity и показывало нужное количество продукта для рецепта) и need_to_buy (сколько единиц продукта надо купить, чтобы хватило для данного рецепта).
Решила проблему n+1 запросов в БД при загрузке списка рецептов - добавила метод setup_eager_loading, убрала поле recipe_nutrients и добавила вместо него аннотированные поля "proteins", "fats", "carbohydrates" и "kcal".
Попыталась решить проблему n+1 запросов в БД при загрузке списка заказов пользователя (История заказов), но пока не удалось, еще в процессе.
Немного сократила число запросов в БД при загрузке списка продуктов (проблема n+1), сделав аннотированные поля rating и is_favorited (если пользователь авторизован).
Подключила OrderingFilter в ProductViewSet, поскольку порядок выдачи продуктов в списке нарушился после появления аннотированного поля rating.
Исправила код, чтобы документация апи в Swagger показывала правильные типы полей в сериализаторах продуктов и рецептов.
Сделала сообщения при логгировании корзины и заказа более подробными.
Исправила в Админке баги при поиске про продуктам, избранным продуктам, отзывам на товары.
Исправила баги id в команде export_data.
Обновила тесты фильтрации продуктов по промоакциям, поскольку фильтрация теперь по полю slug вместо id.

@earlinn earlinn merged commit 8061693 into develop Dec 25, 2023
4 checks passed
@earlinn earlinn deleted the feature/recipe_additional_fields branch December 26, 2023 04:43
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