Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Could you make this module compatible with Domain Access, so that the user can select a different profile for each domain?
See the following discussion for an explanation of how Domain Access support was added to the Google Analytics module:
Post #19 contains a module that adds a Google Analytics account number box to the individual domain settings screens. Would like to have a select box in this screen that would allow the user to select the GA profile he wants to use for each domain.
Comment | File | Size | Author |
---|---|---|---|
#9 | domain-conf-1063532-9.patch | 958 bytes | mallezie |
Comments
Comment #1
mstrelan CreditAttribution: mstrelan commented+1
Comment #2
raspberryman CreditAttribution: raspberryman commentedComment #3
grendzy CreditAttribution: grendzy commentedComment #4
grendzy CreditAttribution: grendzy commentedanyone willing to research this one? Some API docs for Domain Access or code examples would be helpful. #547390: Domain Access & Separate Google Analytics Accounts is not very useful since it contains patches to the Domain Access project itself, and not contrib modules.
Comment #5
aaron1234nz CreditAttribution: aaron1234nz commentedThis code will add both the per-domain settings for Google Analytics Reports. This could easily be rolled into this module. The hook will only ever fire if the domain_access module is installed.
I'm running this code on one of my domain access sites and works a treat.
Comment #6
grendzy CreditAttribution: grendzy commentedCommitted to 7.x:
http://drupalcode.org/project/google_analytics_reports.git/commit/df0b85b
Comment #7
grendzy CreditAttribution: grendzy commentedCommitted to 6.x:
http://drupalcode.org/project/google_analytics_reports.git/commit/52bb42a
Comment #9
mallezieDomain access 3.3 uses hook_domain_conf() for creating this.
Code in Google_analytics reports uses hook_domainconf() (which is obviously not working).
Patch attached.
Comment #10
grendzy CreditAttribution: grendzy commentedThanks for the report. Let's make a separate issue: #1423188: Update to domain access API version 3