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

Omit code specific to Google Analytics #284

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

westonruter
Copy link
Collaborator

The Google Analytics integration should not be part of WordPress core and thus not part of the PWA plugin. To achieve integration with Google Analytics, the code should be added to the service worker by another plugin that is responsible for Google Analytics (e.g. Site Kit).

  • Omit the workbox-offline-ga module.
  • Omit Google Analytics cache name logic from workbox-core
  • Omit Google Analytics from workbox-sw lazy-loading logic.

@westonruter westonruter changed the base branch from master to develop June 19, 2020 19:07
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.

None yet

1 participant