Цели JS событие на кнопку "Оформить заказ"

[santur]

Добрый день! Второй день пытаюсь установить цель JS событие метрики на кнопку "оформить заказ". Делаю так:

<div class="buttons">
  <div class="pull-right">
    <input type="button" value="{{ button_confirm }}" id="button-confirm" data-loading-text="{{ text_loading }}" class="btn btn-primary" ym(66212365,'reachGoal','checkout')
  </div> />
</div>

 

Подскажите, добрые люди - что не так и как правильно?

[Seriusis]

onclick="ym(66212365,'reachGoal','checkout')"

[spectre]

так очень плохо делать

если у человека адблок то ничего не случится, ни заказ, ни событие

 

ну и надо событие делать на оформленный заказ а не на клик по кнопке

[santur]

8 минут назад, spectre сказал:

так очень плохо делать

если у человека адблок то ничего не случится, ни заказ, ни событие

 

ну и надо событие делать на оформленный заказ а не на клик по кнопке

 

Событие на оформленный заказ - это вы имеете в виду просмотр страницы подтверждения заказа?

И чем плохо делать события на клик по кнопке "Оформить заказ"?

[santur]

11 минут назад, spectre сказал:

так очень плохо делать

если у человека адблок то ничего не случится, ни заказ, ни событие

 

ну и надо событие делать на оформленный заказ а не на клик по кнопке

 

А как правильно сделать?)

[Seriusis]

14 минут назад, spectre сказал:

ну и надо событие делать на оформленный заказ а не на клик по кнопке

полностью согласен, лучше ставить цель на страницу checkout/success, в даном случае я указал человеку на его ошибку в коде

[spectre]

25 минут назад, santur сказал:

 

А как правильно сделать?)

правильно так 

на странице checkout success проверить есть ли order_id (чтобы избежать дублей цели при обновлении страницы) 

если есть то вывести код цели, предварительно проверив что ym это функция