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

Fully remove Google Analytics classic #182

Merged
merged 1 commit into from May 28, 2015
Merged

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented May 26, 2015

Following on from alphagov/govuk_frontend_toolkit#194 and #180

  • Bump govuk_frontend_toolkit to pull in latest analytics
  • Remove references to classic tracker
  • Remove classic specific arguments (eg custom variable name and scope)
  • Update references of GOVUK.Tracker to GOVUK.Analytics
* Bump frontend_toolkit to pull in analytics changes
* Remove references to classic tracker
* Rename GOVUK.Tracker to GOVUK.Analytics
@fofr
Copy link
Contributor Author

@fofr fofr commented May 28, 2015

As Trade Tariff frontend has already switched to Universal only, there's nothing holding up this PR from being merged (if people are happy) and deploying.

bradwright added a commit that referenced this pull request May 28, 2015
Fully remove Google Analytics classic
@bradwright bradwright merged commit db0d0d7 into master May 28, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@bradwright bradwright deleted the remove-classic-analytics branch May 28, 2015
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.