Skip to content

Add promo code (coupon) support to order form - #615

Open
ninedev-i wants to merge 6 commits into
mainfrom
feature/promocode
Open

ninedev-i wants to merge 6 commits into
mainfrom
feature/promocode

Conversation

@ninedev-i

@ninedev-i ninedev-i commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Что происходит; кому и зачем нужно:
поле выбора кто может использовать промокод
перенес поле ввода промокода из корзины на страницу оформления заказа
бэк


Что поменялось для пользователей:
image


Как проверял/а:
тестинг


Чеклист ревью:

@ninedev-i
ninedev-i requested a review from koledennix October 6, 2026 09:31
@ninedev-i
ninedev-i requested a review from a team as a code owner October 6, 2026 09:31
@codacy-production

codacy-production Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 98 high · 2 medium

Alerts:
⚠ 100 issues (≤ 0 issues of at least minor severity)

Results:
100 new issues

Category Results
BestPractice 2 medium
4 high
ErrorProne 89 high
Security 5 high

View in Codacy

🟢 Metrics 65 complexity

Metric Results
Complexity 65

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@ekateluv

Copy link
Copy Markdown
Contributor

Может быть нам не писать итого, пока не рассчитана стоимость доставки? Может вместо знака вопроса написать серым "рассчитайте доставку" (или что то подобное, более осмысленное), поместится?
изображение
Вот тут обрезается начало строки имени, думаю принято наоборот показывать начало и скрывать окончание
изображение

@ekateluv

ekateluv commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Ввела случайный существующий адрес в беларуси, но не вывел ни ошибку ни сумму доставки, непонятно что произошло
изображение

@ekateluv

Copy link
Copy Markdown
Contributor

Не знаю, можно ли было так раньше сделать, но если поискать страну в поиске и нажать enter то сохранится пустое значение поля страна и список способов доставки никак не реагирует на это
изображение

Comment thread pages/contents/order.vue Outdated
'cdek_pvz_id', 'cdek_pvz_address', 'cdek_pvz_country_code',
'cdek_pvz_city_code', 'cdek_pvz_city', 'cdek_pvz_tariff', 'cdek_pvz_postal_code',
];
const normalizeDeliveryData = (value: Record<string, any>) =>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А нельзя ли это выделить в отдельный файлик? А то все в кучу сваливается :(

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@ninedev-i
ninedev-i requested a review from ekateluv October 22, 2026 09:36
@ekateluv

Copy link
Copy Markdown
Contributor

Что то все равно не то с доставкой. Вот я пробую этот адрес на тестовом и проде
изображение
изображение

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