Skip to content
This repository has been archived by the owner. It is now read-only.

Track contents clicks #131

Merged
merged 1 commit into from Feb 6, 2017
Merged

Track contents clicks #131

merged 1 commit into from Feb 6, 2017

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Feb 6, 2017

  • Match government-frontend category, action and label
  • Increment heading_level each time partial gets nested, to allow tracking of H2, H3 and H4
  • Fix 3-space whitespace

Goes with alphagov/government-frontend#235
Part of https://trello.com/c/62Gy8pyj/77-track-interactions-with-contents-lists-1-s

Example clicks tracked:
screen shot 2017-02-06 at 10 53 15

@nickcolley

* Match government-frontend category, action and label
* Increment heading_level each time partial gets nested, to allow
tracking of H2, H3 and H4
* Fix 3-space whitespace
@nickcolley
Copy link
Contributor

@nickcolley nickcolley commented Feb 6, 2017

I'm getting
screen shot 2017-02-06 at 11 24 11

@nickcolley
Copy link
Contributor

@nickcolley nickcolley commented Feb 6, 2017

Just made sure static is up-to-date and that was the issue this looks good to me 👍

@nickcolley nickcolley merged commit 7c9e499 into master Feb 6, 2017
1 check passed
1 check passed
continuous-integration/jenkins/branch This commit looks good
Details
@nickcolley nickcolley deleted the track-contents-clicks branch Feb 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.