Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Adding support to supply timestamp #2

Merged
merged 2 commits into from Sep 27, 2011

Conversation

Projects
None yet
2 participants
Contributor

kenperkins commented Aug 23, 2011

My company processes server side events asynchronously and so I needed to supply the actual timestamp in the event.

Should be pretty straightforward.

--ken

carlsverre added a commit that referenced this pull request Sep 27, 2011

Merge pull request #2 from kenperkins/master
Adding support to supply timestamp -- Thanks Ken Perkins!

@carlsverre carlsverre merged commit e9017e2 into mixpanel:master Sep 27, 2011

Contributor

carlsverre commented Sep 27, 2011

Sorry about the wait btw, I wasn't notified by GH about this pull request. _

Contributor

kenperkins commented Sep 27, 2011

No biggie. As it turns out of course, mixpanel ignores timestamps greater than 48 hours so it turned out to not really matter.

--Ken
clipboard, inc.
we're hiring!: www.clipboard.com/jobs

On Tuesday, September 27, 2011 at 12:10 PM, Carl Sverre wrote:

Sorry about the wait btw, I wasn't notified by GH about this pull request. _

Reply to this email directly or view it on GitHub:
carlsverre#2 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment