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

When a campaign changes, creates a new "visit" by default #2624

Closed
mattab opened this issue Aug 12, 2011 · 7 comments
Closed

When a campaign changes, creates a new "visit" by default #2624

mattab opened this issue Aug 12, 2011 · 7 comments
Assignees
Milestone

Comments

@mattab
Copy link
Member

@mattab mattab commented Aug 12, 2011

In case you are using intra website campaign links, or if the visitor comes back on your site from a different website and is registered as a different campaign, we could automatically create a new visit.

There would be only 1 unique visitor, but 2 visits, if the visitor comes back to the site less than 30 min ago with the same browser but a different campaign parameters combination.

Proposed steps

  • implemented as a setting, enabled by default: tracker_create_new_visit_when_campaign_changes
  • we could also have the same setting for websites
    tracker_create_new_visit_when_website_referrer_changes
    Enabling these 2 settings would mean that all referrer data would be tracked.
  • maybe also available via a JavaScript option? (what does GA offer here and should we do similar?)
    Visits will be slightly increased but this should be really insignificant.

Conclusion

This change would result in more accurate tracking, better campaign attribution. IT would also allow to use Piwik to do internal campaign tracking which would be a very useful feature to have for some websites.

See also #2567 and #2468. Note: this was discussed a few times in the forums and was implemented in GA recently.

@mattab mattab removed P: normal labels Aug 3, 2014
@mattab mattab added the Major label Sep 15, 2014
@diosmosis diosmosis self-assigned this Dec 9, 2014
diosmosis added a commit that referenced this issue Dec 10, 2014
…paign/website referrer information changes between actions. Includes new tracker hook shouldForceNewVisit that dimensions can use to force a new visit during tracking. Also includes light refactoring to referrer dimensions. Testless.
@diosmosis

This comment has been minimized.

Copy link
Member

@diosmosis diosmosis commented Dec 12, 2014

Fixed by #6847

@diosmosis diosmosis closed this Dec 12, 2014
@asafyish

This comment has been minimized.

Copy link

@asafyish asafyish commented Jan 1, 2015

How would this affect AdvancedCampaignReporting ?
As of now, When a visitor comes back (within 30 minutes) with different utm_source, there is no new visit, and the original utm_source remains. Will this change that behaviour ? (I hope so)

@mattab

This comment has been minimized.

Copy link
Member Author

@mattab mattab commented Jan 5, 2015

How would this affect AdvancedCampaignReporting ?

This does not affect the AdvancedCampaignReporting plugin actually. Could you create an issue in the repo there? we could make the INI setting work for AdvancedCampaignReporting too with some extra work

@diosmosis

This comment has been minimized.

Copy link
Member

@diosmosis diosmosis commented Jan 5, 2015

@mattab It should affect AdvancedCampaignReporting, I remember changing the expected test results.

@mattab

This comment has been minimized.

Copy link
Member Author

@mattab mattab commented Jan 5, 2015

It doesn't really work though, because the logic in isReferrerInformationNew reads the fields array('referer_keyword', 'referer_name', 'referer_type') but AdvancedCampaignReporting stores the campaign info in other campaign_* fields

@diosmosis

This comment has been minimized.

Copy link
Member

@diosmosis diosmosis commented Jan 5, 2015

@mattab Oh I remember, in Tracker::addDimensionsToRow, it sets the referer_name/referer_type/referer_keyword, I guess that's enough to make it work for AdvancedCampaignReporting? Or maybe just in the case of the test.

@mattab

This comment has been minimized.

Copy link
Member Author

@mattab mattab commented Feb 17, 2015

Added FAQ to document this feature: https://piwik.org/faq/how-to/faq_19616/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.