-
Notifications
You must be signed in to change notification settings - Fork 2
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 #248
Merged
juliana-str
merged 9 commits into
develop
from
fix/shoppingcart_product_photo_full_paths
Feb 6, 2024
Merged
Fix/shoppingcart product photo full paths #248
juliana-str
merged 9 commits into
develop
from
fix/shoppingcart_product_photo_full_paths
Feb 6, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
earlinn
commented
Feb 3, 2024
•
edited
Loading
edited
- Исправила путь к тестам pytest во время выполнения workflow, раньше pytest в GitHub Actions запускал тесты django.test (unittest) тоже.
- Исправила баг промокода (был нечувствителен к регистру, стал чувствителен).
- Сделала поле promotion_type в CouponSerializer в виде read_only, теперь туда всегда автоматически методом perform_create подставляется "coupon", поэтому метод validate_promotion_type больше не нужен.
- Исправила баг с возрастом - добавила на уровень апи проверку возраста пользователя (не менее 6 лет, не болеем 110 лет), оказалось, что метод clean_fields модели User работал только в Админке, а на уровне апи не вызывался, поэтому можно было ставить всякие дурацкие даты рождения.
- Усовершенствовала проверку возраста в методе clean_fields модели User: раньше там проверялся только год, поэтому система пропускала слишком молодых пользователей, если нужный год наступил, а их ДР еще не наступило, либо слишком старых, а теперь все точно день в день, были магические цифры (убрала в переменные).
- Добавила тесты на возраст пользователя - на слишком юный и слишком старый.
- Исправила тест на обнуление даты рождения (он не проходил и пропускался, теперь проходит, а в комментарии я описала, почему он раньше не проходил).
- Запретила в Админке применять промокоды к уже созданным заказам - сделала поля "coupon_applied" и "coupon_discount" заказа read_only (на Багатоне решили, что невозможность применить промокод к заказу - это баг, а это на самом деле фича, но они не могли этого узнать).
- Исправила баг фильтрации продуктов по категории, в которой нет ни одного продукта: раньше даже если сама категория существовала, но в ней не было продуктов, фильтр отвечал статусом 400 (validation error), теперь он выдает в ответ на это пустой список, как и в случае фильтрации по несуществующей категории. Аналогичным образом поменяла фильтры для подкатегорий, компонентов, производителей, тегов и промоакций - иначе там возникли такие же ошибки. Проапдейтила тесты в соответствии с внесенными изменениями.
- Добавила новые поля, сортировки и инлайны в Админке:
- категорий (добавила отображение количества продуктов для каждой категории, добавила сортировку категорий по количеству продуктов и по количеству подкатегорий),
- подкатегорий (добавила отображение количества продуктов для каждой подкатегории, добавила сортировку подкатегорий по количеству продуктов),
- компонентов (добавила отображение количества продуктов для каждого компонента, добавила сортировку компонентов по количеству содержащих их продуктов, добавила отображение на странице компонента всех продуктов, в которых он содержится),
- тегов (добавила отображение количества продуктов для каждого тега, добавила сортировку тегов по количеству продуктов),
- производителей (добавила отображение количества продуктов данного производителя, добавила сортировку производителей по количеству продуктов),
- промоакций (добавила отображение количества продуктов для каждой промоакции, добавила сортировку промоакций по количеству продуктов, добавила отображение на странице промоакции всех продуктов, к которым она применена),
- промокодов (добавила отображение количества заказов для каждого промокода, добавила сортировку промокодов по количеству заказов, добавила отображение на странице промокода всех заказов, к которым он был применен),
- продуктов (добавила отображение количества отзывов и добавлений в избранное для каждого продукта, добавила сортировку продуктов по количеству добавлений в избранное, по количеству отзывов и по рейтингу, добавила на странице продукта отображение всех отзывов на него)
…PI level, update this check at the model level
…ts from django.test during GitHub Actions workflow
… workflow execution
…h date format error message
…, tags and promotions; add new fields, ordering options and inlines to the Admin panel for categories, subcategories, components, tags, producers, promotions, coupons and products; update tests
juliana-str
approved these changes
Feb 6, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.