Skip to content

Conversation

@rcs
Copy link
Contributor

@rcs rcs commented Jul 24, 2013

Adds support for setPlatform in the javascript tracker. Example use case here would be html+js running inside node-webkit, or a webview on a mobile platform.

@alexanderdean
Copy link
Member

Nice! Thanks Ryan. Scheduling for 0.13.0...

@rcs
Copy link
Contributor Author

rcs commented Jul 25, 2013

This raises an issue re: https://github.com/snowplow/snowplow/wiki/snowplow-tracker-protocol#2-platform-specific-parameters . There we're saying those things are only feasible for the "web" platform, but all the rest of the "html-js medium" platforms would make sense to have those values too.

@alexanderdean
Copy link
Member

Yes - we have to be a bit careful here. We need to decide if we want setPlatform to be an informational thing or to proscribe what parameters are available/expected on this platform.

A webview on a mobile platform is a good example of this: our current thinking is that the platform should still be "web", because all of the context available to a web browser is available to this event. But this isn't very informational if you know this tracker will only be used on mobile devices...

/cc @yalisassoon

@alexanderdean alexanderdean merged commit 18ec8af into snowplow:master Jan 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:context A new context entity, or changing an existing one, is required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants