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

Fix/shoppingcart product photo full paths #247

Merged
merged 14 commits into from
Feb 1, 2024

Conversation

earlinn
Copy link
Collaborator

@earlinn earlinn commented Feb 1, 2024

  • Обновила и дополнила разделы README, касающиеся запуска проекта в docker-контейнерах и пользователей.
  • Поставила требование, чтобы все номера заказов были уникальными (на уровне модели Order), создала файл миграции
  • Сделала в функции generate_order_number проверку на существование в БД заказа с подобным номером, а если такой заказ уже есть, то добавление к генерируемому номеру заказа еще одного случайного числа.
  • Переименовала старый CouponSerializer (используется во время применения промокода к стоимости Корзины) в CouponApplySerializer, поля остались без изменений, только название поменялось.
  • Создала новый CouponSerializer и сделала эндпойнты апи для просмотра (доступно всем), редактирования и удаления (доступно админам) промокодов: /api/coupons/ и /api/coupons/{id}/
  • Запретила применять промокод к конкретным продуктам, в этом случае выводится сообщение о том, что промокод можно применять только к Корзине в целом
  • Сделала в Корзине вывод ссылки на фото продукта таким же, как у других эндпойнтов (было "photo": "images/products/50.jpg", стало "photo": "http://127.0.0.1:8000/media/images/products/50.jpg")
  • Добавила поле pub_date в эндпойнты рецептов
  • Дополнила команды export/load data моделью Coupon, а также внесла в уже существующие модели экспорт/загрузку новых полей: "coupon_applied_id" и "coupon_discount" в модели Order, "short_description" и "servings_quantity" в модели Recipe

@juliana-str juliana-str merged commit 31daad0 into develop Feb 1, 2024
4 checks passed
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