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.

Sign up for GitHub

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

Jump to bottom

trackStructuredEvent() misleading docs #106

Closed
n-belokopytov opened this issue Mar 23, 2015 · 1 comment
Closed

trackStructuredEvent() misleading docs #106

n-belokopytov opened this issue Mar 23, 2015 · 1 comment
Assignees
@jbeemster
Labels
category:documentation About the documentation. type:defect Bugs or weaknesses. The issue has to contain steps to reproduce.
Milestone
Version 0.4.0

Comments

@n-belokopytov
Copy link

https://github.com/snowplow/snowplow/wiki/Android-Tracker#45-track-structured-events-with-trackstructuredevent
reads
"up to five fields (with only the first two required)"
While in fact all five are required by Snowplow and it's shown in the table below the aforementioned line.
@alexanderdean alexanderdean added type:defect Bugs or weaknesses. The issue has to contain steps to reproduce. category:documentation About the documentation. labels Mar 23, 2015
@alexanderdean alexanderdean added this to the Version 0.4.0 milestone Mar 23, 2015
@n-belokopytov
Copy link
Author

Empty strings don't work with them either, so we had to resort to defaulting them to "default"

jbeemster added a commit that referenced this issue Jun 5, 2015
@jbeemster
Fixed trackStructuredEvent() misleading docs (closes #106)
ad81e1e 
This function should only require the first two fields, has been fixed
so that everything else can be passed as ‘null’.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jbeemster jbeemster

Labels
category:documentation About the documentation. type:defect Bugs or weaknesses. The issue has to contain steps to reproduce.
Projects
None yet
Milestone
Version 0.4.0
Development

No branches or pull requests
3 participants
@alexanderdean @n-belokopytov @jbeemster