Skip to content
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

Added setting for disabling Heatmap session recordings #61

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

emiltorp
Copy link

@IreneTepe
Copy link

Hello! :)
Do we have an update regarding this REALLY helpful change?
If it is going to be merged or...
Thank you in advance :)

@Arnaud73
Copy link
Owner

Arnaud73 commented Jul 8, 2022

I started finishing ngx-matomo for Ng14 last week.

I also saw your PR and decided that instead, I would integrate all Heatmap functions for the tracker. Code is there (on my local branch), however, my last concern is to decide how to name all those functions since some have names (in the Matomo tracker) that collide with others if not prefixed by the name of the Matomo module (I intend to use the same naming convention for features for Media Analyctics, Form Analytics and so on).

Si stay tuned, it shall not be long!

@Arnaud73 Arnaud73 self-assigned this Jul 8, 2022
@Arnaud73 Arnaud73 added the enhancement New feature or request label Jul 11, 2022
@emiltorp
Copy link
Author

I'm guessing, since you added this to ngx-matomo v1, we can close this PR? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants