search

ragnarpeterson / ember-cli-intercom

ember addon to easily add Intercom to your app
MIT License
10 stars 15 forks source link

Add support for Intercom('trackEvent') #5

Closed olivierobert closed 8 years ago

olivierobert commented 8 years ago

Continuing the work of @andreicek, this PR adds support for event tracking.

While implementing a user on-boarding, I was faced with issues using url-based condition. Events works better in most cases in single page apps.

FYI: currently, Intercom('update') is called on every page transition but I was informed by Intercom support that there is a SPA limit of 10 calls per 30 minutes. As a consequence, "last viewed" does not always reflect the last page the user was on. Hence, using events has been reliable for me.

jasonkriss commented 8 years ago

@olivierobert LGTM! Thanks a lot! I did not realize that was the case with the Intercom('update') call. Good to know!