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

Add link tracking to left-hand contents list #235

Merged
merged 2 commits into from Jan 27, 2017
Merged

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Jan 20, 2017

  • Move contents logic into extracts headings module

    • Provide new helper methods for contents_items and contents_link allowing presenters to override them if needed
    • Override content_items method in collections to use a list of groups
    • Add extra headings to working groups when appropriate
  • Add link tracking module and attributes to all contents links

Category: contentsClicked
Action: leftColumnH2
Label: ID of the heading

Depends on alphagov/static#883

https://trello.com/c/62Gy8pyj/77-track-interactions-with-contents-lists-1-s

cc @nickcolley

@boffbowsh boffbowsh temporarily deployed to government-frontend-pr-235 Jan 20, 2017 Inactive
@fofr fofr changed the title [Do not merge] Add link tracking to left-hand contents list Add link tracking to left-hand contents list Jan 27, 2017
@gpeng
gpeng approved these changes Jan 27, 2017
fofr added 2 commits Jan 20, 2017
* Move contents logic into extracts headings module
* Provide new helper methods for contents_items and contents_link
allowing presenters to override them if needed
* Switch to new module
* Override content_items method in collections to use a list of groups
* Add extra headings to working groups when appropriate
Use `contentsClicked` category to match `breadcrumbClicked` and
`relatedItemClicked` in static.

* Indicate that the action is a click on a content item in a left column
* Provide the link’s ID as the label
@fofr fofr force-pushed the track-contents-link-clicks branch from c159501 to f774352 Jan 27, 2017
@fofr fofr merged commit dc33a1f into master Jan 27, 2017
1 check passed
1 check passed
continuous-integration/jenkins/branch This commit looks good
Details
@fofr fofr deleted the track-contents-link-clicks branch Jan 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

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