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

COSBETA-80: Set up analytics #759

Merged
merged 2 commits into from
Mar 15, 2019
Merged

Conversation

kicferk1
Copy link
Contributor

This changes the google analytics id we use so that cosmetics is its own property.

The reason to make it its own property is to make tracking user journeys in keycloak easier. Adding correct tags to keycloak was done on a separate pull request and seems to be working as intended.

Checklist:

  • Automated checks are passing locally.

General testing

  • Test without javascript(pretty sure it won't work with js disabled though)

Copy link
Contributor

@tom-bell-softwire tom-bell-softwire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@tom-bell-softwire tom-bell-softwire merged commit d7752b3 into master Mar 15, 2019
@tom-bell-softwire tom-bell-softwire deleted the COSBETA-80_set_up_analytics branch March 15, 2019 14:13
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.

2 participants