Skip to content

etracker wrapper function added for single page applications compatibility #240

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

Merged
merged 7 commits into from
Apr 23, 2020

Conversation

ben-dw
Copy link
Contributor

@ben-dw ben-dw commented Apr 22, 2020

I have added one further function of etracker (https://www.etracker.com/docs/integration-setup/tracking-code-sdks/tracking-code-integration/dynamische-inhalte-wrapper/).
This function is needed for SPA (single page applications) to trigger pageviews.

@tsteur
Copy link
Member

tsteur commented Apr 23, 2020

Cheers @ben-dw

@tsteur tsteur merged commit 853a2c3 into matomo-org:4.x-dev Apr 23, 2020
@ben-dw
Copy link
Contributor Author

ben-dw commented Apr 28, 2020

I'm wondering why the changelog 3.13.5 shows that this ticket is integrated. I can't see any changes in the download file for the new matomo version.
@tsteur : Do you have any idea?

@sgiehl
Copy link
Member

sgiehl commented Apr 28, 2020

That actually shouldn't be included in the changelog. It was merged to 4.x-dev branch, so it will be included in the first Matomo 4 release and not in any 3.x release. ping @mattab

@sgiehl sgiehl added this to the Matomo 4 milestone Apr 28, 2020
@Findus23
Copy link
Collaborator

@mattab
Copy link
Member

mattab commented Apr 30, 2020

now deleted the entries from changelog, thanks for the notice @ben-dw

(also renamed the milestone to Matomo 4.0.0 so it would match my regex I had set: 4.0*)

@mattab mattab changed the title etracker wrapper function added etracker wrapper function added for single page applications compatibility Sep 29, 2020
@ben-dw ben-dw deleted the patch-5 branch March 26, 2021 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants