Code Monkey home page Code Monkey logo

Comments (13)

narkq avatar narkq commented on August 18, 2024

Пожалуйста, отформатируйте вывод по-нормальному.
Ну или скриншот приложите, на худой конец 🙂
Кроме того, не вижу в Вашем сообщении собственно текста ошибки.
Когда именно возникает ошибка? При сборке webpack-ом? При открытии страницы?

from react-yandex-metrika.

romanown avatar romanown commented on August 18, 2024

в момент компиляции такое. TypeError: window[_constants.accountListName] is undefined. я это из браузера скопировал что в сообщении

from react-yandex-metrika.

romanown avatar romanown commented on August 18, 2024

`TypeError: window[_constants.accountListName] is undefined
ymAsyncProxy
/node_modules/react-yandex-metrika/lib/index.js:41

args[_key2] = arguments[_key2];

}

window[_constants.accountListName].forEach(function (id) {
var trackerVersion = window[(0, _constants.trackerVersionName)(id)];
var callbackQueue = window[(0, _constants.callbackQueueName)(trackerVersion)];
if (callbackQueue) {

ym
/node_modules/react-yandex-metrika/lib/index.js:56

function ym() {
if (isBrowser) {
ymAsyncProxy.apply(undefined, arguments);
}
}

./src/RoutesContainer.js/webpack_exports.a<
D:/npm/dress/dress/src/RoutesContainer.js:70

if (typeof window !== 'undefined') { ym('hit', props.location.pathname) } `

from react-yandex-metrika.

romanown avatar romanown commented on August 18, 2024

но при переносе кода в другие места эта же ошибка выходила при попытке перехода между страницами сайта. используется серверный рендеринг.

from react-yandex-metrika.

narkq avatar narkq commented on August 18, 2024

Я правильно понимаю, что RoutesContainer.js - это компонент роутера?
Не совсем ясно, что там за if-statement в перемешку с JSX?

Вам нужно по сути написать код, который сможет реагировать на изменения роута и посылать соответствующие события. Это ни в коем случае нельзя делать в методе render. Как правило, это делают в методе componentWillReceiveProps (вот как например в библиотеке, ссылку на которую Вы давали https://github.com/sonaye/react-with-analytics/blob/master/src/withAnalytics.js#L23 ). Впрочем, componentWillReceiveProps вроде как deprecated, так что может быть можно просто подписаться на события навигации через https://github.com/ReactTraining/history например.

from react-yandex-metrika.

romanown avatar romanown commented on August 18, 2024

наверное надо было полностью привести код как оно сейчас.
import { YMInitializer } from 'react-yandex-metrika'; import ym from 'react-yandex-metrika'; const HistoryListener = withRouter(class extends React.Component { componentDidMount() { this.props.history.listen((props) => { ym('hit', this.props.location.pathname); }) } render() { return null } }) export default withRouter(props => { return ( <div> <YMInitializer accounts={[49433788]} /> <HistoryListener /> <Switch> <Route exact path="/" component={Home} />

from react-yandex-metrika.

narkq avatar narkq commented on August 18, 2024

Так все-таки в состоянии "как оно сейчас" работает? Или такая же ошибка?

Если такая же ошибка, то скорее всего это связано с тем, что к тому времени, как срабатывает HistoryListener.componentDidMount, не успевает отработать YMInitializer.componentDidMount (который, собственно, и создает глобальные объекты типа window[_constants.accountListName]).

Чтобы такого не происходило, можно поместить YMInitializer внутрь HistoryListener-а (т.к. метод componentDidMount вызывается в порядке, обратном иерархии компонентов). Например, так:

const HistoryListener = withRouter(
  class extends React.Component {
    historyUnlisten = null

    componentDidMount() {
      this.historyUnlisten = this.props.history.listen((location, action) => {
        ym('hit', `${location.pathname}${location.search}${location.hash}`)
      })
    }

    componentWillUnmount() {
      this.historyUnlisten()
    }

    render() {
      return <YMInitializer accounts={[49433788]} />
    }
  }
)

export default withRouter(
  props => {
    return (
      <div>
        <HistoryListener />
        <Switch>
          <Route exact path="/" component={Home} />
          ...
        </Switch>
        ...
      </div>
    )
  }
)

from react-yandex-metrika.

narkq avatar narkq commented on August 18, 2024

ибо дидмоунт вызывается не при всех переходах

Ох. Он и не должен вызываться при всех переходах. Нужно сделать так, чтобы componentDidMount для компонентов HistoryListener и YMInitializer вызывался (желательно) ровно один раз на полную перезагрузку страницы. После этого при навигации через роутер должен вызываться только обработчик, переданный в history.listen.

from react-yandex-metrika.

romanown avatar romanown commented on August 18, 2024

теоретически так оно и есть. поскольку что-то показывается, то инициализация идет. а вот при переходе уже не срабатывает передача hit. хотя код туда доходит. я ставил алерт на каждое место.

from react-yandex-metrika.

narkq avatar narkq commented on August 18, 2024

Какая версия react-yandex-metrika подключена?
Возможно, я кое-что сломал в 2.4.0 (или оно вообще не работало для случая, когда серверный код собирается webpack-ом).

from react-yandex-metrika.

romanown avatar romanown commented on August 18, 2024

"version": "2.3.0"

from react-yandex-metrika.

narkq avatar narkq commented on August 18, 2024

Попробуйте поставить из вот этой ветки https://github.com/narkq/react-yandex-metrika/tree/fix-bundler-eval (например, просто удалить node_modules/react-yandex-metrika и склонировать туда эту ветку).

Вызов ym('hit') по идее нужен только в обработчике, который слушает события навигации.
При первоначальной загрузке страницы сама библиотека яндекс метрики отсылает hit (если только не указано defer: true при подключении счетчика).

from react-yandex-metrika.

romanown avatar romanown commented on August 18, 2024

после обновления все работает. спасибо. это единственный на данный момент найденный мною компонент модуль для react reactjs для отправки сообщения в яндекс метрику о совершенном переходе ya.metrika hit для SPA сайта. написал так чтобы другие нашли эту страницу.

from react-yandex-metrika.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.