search

vchaptsev / vue-yandex-metrika

Vue plugin for Yandex Metrika
MIT License
73 stars 34 forks source link

Seems a bug in 1.8.0 #14

Closed irriss closed 5 years ago

irriss commented 5 years ago

Hello,

Thanks for this package.

After upgrading to 1.8.0 got this bug in console

Uncaught SyntaxError: Unexpected token < vue-yandex-metrika.js?932e:1 Uncaught ReferenceError: Ya is not defined at u (vue-yandex-metrika.js?932e:1) at HTMLScriptElement.eval (vue-yandex-metrika.js?932e:1)

1.7.3 does not have this issue.

Could you help fixing it?

irriss commented 5 years ago

actually the error is gone if I add new param explicitly

scriptSrc: 'https://mc.yandex.ru/metrika/tag.js'

strange that default value isnt picked up

konart commented 5 years ago

This one should be either a documented feature or an issue to be fixed.

cranx commented 5 years ago

+1 Why does scriptSrc should be an option? I guess https://mc.yandex.ru/metrika/tag.js must be default value at least

Bargamut commented 5 years ago

You can read this. It's not a bug, it's a feature. )

But it will be better if this option will have value by default, of course.

vchaptsev commented 5 years ago

Fixed by @0311snitch here: #15 New version 1.8.1 is available.