-
Notifications
You must be signed in to change notification settings - Fork 0
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
Несколько моментов для улучшения плагина. #5
Comments
@gleb-pineparks первый и второй момент возьмем на проработку
Это все делается в личном кабинете ApiShip с помощью функционала "Редактор тарифов" |
Спасибо! На такой волне, попрошу об ещё о одном добавлении :) Прошу добавить перед "mapApi.ajax(request)", тригер события, напримервот так: Назвать событие можете подругомы, это как пример, главное передавать по нему тоже самое что и в ajax идёт :) У нас к примеру, очень нестандартная страница оформления заказа и нам нужно знать адрес выбранного места ПВЗ, чтобы прописать его в поле адрес. |
@apiship-team И ещё одно :) Ниже в примере, строчка выделена жирным, оберните текст кнопки в филтр, на плагин это не повлияет визуално никак, а нам и другим позволит убрать оттуда скобки и показывать как нормалную стилизованую кнопку :) if ( in_array( Options\WP_ApiShip_Options::DELIVERY_TO_POINT_OUT, $tariff->deliveryTypes ) ) { Либо весь $elem обернуть в apply_filters, будет даже лучше :) Спасибо! |
@gleb-pineparks взяли в работу |
Первый момент:
Измените обпащение к "Городу", не всегда он именно "input", у нас например "select", лучше просто по ID брать значение
if ( differentCity ) { city = $('#billing_city').val(); } else { city = $('#billing_city').val(); }
Второй момент:
Отложите обновление страницы оформления до момента как AJAX "saveClientSelectedPoint" закончит работу, например:
mapApi.ajax(request).done(function() { $('body').trigger('update_checkout'); });
Иначе название выбраного ПВЗ сбрасывается на предыдущее значение.
Третий момент:
Добавте возможность какьто групировать или скрывать методы доставки курьером, иначе вылазит огромный список со всеми возможными вариантами...
Можно сделать по примеру как у вас вывот ПВЗ реализован ("Отдельный способ доставки до ПВЗ для каждого тарифа", "Отдельный способ доставки до ПВЗ для каждой СД", "Все ПВЗ на одной карте")
The text was updated successfully, but these errors were encountered: