Skip to content
This repository has been archived by the owner on Feb 11, 2020. It is now read-only.

virtual page tracking still uses #page-n #318

Closed
ScottDowne opened this issue Aug 26, 2015 · 5 comments
Closed

virtual page tracking still uses #page-n #318

ScottDowne opened this issue Aug 26, 2015 · 5 comments
Assignees

Comments

@ScottDowne
Copy link
Contributor

@adamlofting thoughts?

In #283 we' re re adding the virtual page tracking. We also are moving away from the #page-n concept.

The tracking is still assuming this and not sure the implications of just passing in a new type of data.

Would be nice to not need to pass in the page number as #page-n as that might be confused with navigation.

@adamlofting
Copy link
Contributor

The #page-n format isn't important in GA. We can pass in any value, so long as each page gets a unique and identifiable string. 😄

@adamlofting adamlofting modified the milestones: Sep 11 - Feature complete for DE, Sep 11, Not yet assigned to a heartbeat Aug 27, 2015
@adamlofting adamlofting added the P1 label Aug 28, 2015
@ScottDowne ScottDowne self-assigned this Aug 28, 2015
@ScottDowne
Copy link
Contributor Author

@adamlofting

I'm wondering if this should be an event, and not a page view?

Feels weird to me to pass it a url that isn't actually the url.

Examples that feel off to me:
/en-US/page-one
/en-US/#page-1
/en-US/?page=1

They all have the same issue, they are not real and we're hacking in data for the purpose of tracking an event, essentially.

I'm thinking:

ga.event( { category: 'Navigation',
                  action: 'Page Navigation',
                  value: 2 } );

@adamlofting
Copy link
Contributor

The benefit of the pageview in GA is you can build funnel reports to show drop-off at each step.

The event data on the other hand isn't as easy to review in a sequential flow as the pageviews.

@ScottDowne
Copy link
Contributor Author

OK, going to leave this as is for now then. Moving to a P2 as this'll just reduce confusion in the code but not actually provide a direct benefit to a user.

@ScottDowne ScottDowne added P2 and removed P1 labels Sep 4, 2015
@ScottDowne
Copy link
Contributor Author

This just doesn't seem to be creating any confusion.

Closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants