From what I can tell, the module installs the tracking code globally for everyone, including logged-in users such as site maintenance individuals, which will ultimately skew the true results. IMHO, there needs to be a way enable tracking by role.

Thanks!

CommentFileSizeAuthor
#4 add-tracking-by-role-1976580-4.patch2.48 KBkyletaylored
#2 crazyegg_exclude_roles-1976580.patch2.57 KBjaydub
#1 crazy_egg_exclude_roles.patch2.91 KBk-mo

Comments

k-mo’s picture

k-mo commented
Issue summary: View changes
StatusFileSize
new crazy_egg_exclude_roles.patch2.91 KB

Patch in attachment will allow you to select roles that wont have crazy_egg tracking.

jaydub’s picture

jaydub commented
StatusFileSize
new crazyegg_exclude_roles-1976580.patch2.57 KB

My company had a use for this patch so I've taken a look and made some revisions here and there. There is another patch in another issue that addes a module .install file to handle deleting variables on module uninstall and since this patch adds a variable the patch should also unset on module uninstall but since the patch to add the module .install is not committed I won't update the patch here for that, rather this issue should be tracked to roll in the variable_del when that other patch is committed.

jaydub’s picture

jaydub commented
Status: Active » Needs review
kyletaylored’s picture

kyletaylored commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new add-tracking-by-role-1976580-4.patch2.48 KB

I tested the patch, but had to reroll it based on recent updates. In addition to rerolling, I fixed the variable_del in the hook_uninstall to capture the latest variables that have been added.

After these fixes, the role based loading seems to still be working!

  • crazyegg committed ec4d3e9 on 7.x-1.x 
    Issue #1976580 by philsward: Add tracking code by role
Applied patch (...
crazyegg’s picture

crazyegg commented

Thank you everybody, we've applied the final version of patch with some modifications. It will be available soon in the official release.

crazyegg’s picture

crazyegg commented
Status: Reviewed & tested by the community » Fixed
crazyegg’s picture

crazyegg commented
Status: Fixed » Closed (fixed)