search

michaelcarter / mixpanel-data-export-js

Mixpanel data export js
91 stars 38 forks source link

maybe csv format support? #8

Closed areagray closed 9 years ago

areagray commented 10 years ago

Hi. Looks like the event csv format parameter causes it to error out, json works like a charm. Would be super to have that support since Mix is already doing the work.

http://d.pr/i/1aJ4i

Thanks for considering - very helpful lib.

michaelcarter commented 10 years ago

Thanks for the suggestion @areagray. An oversight on my part no doubt. I have some free time this weekend so might be able to look at getting this done then. :-)

michaelcarter commented 9 years ago

Hi @areagray. Just to let you know I've not forgotten about this, full time job/life etc getting in the way. I did some work on the repo last weekend but that was mainly adding promise support.

areagray commented 9 years ago

Hey thanks Mike. Understood. Thanks for the helpful module.

On Thu, Dec 4, 2014 at 1:14 AM, Mike Carter notifications@github.com wrote:

Hi @areagray https://github.com/areagray. Just to let you know I've not forgotten about this, full time job/life etc getting in the way. I did some work on the repo last weekend but that was mainly adding promise support.

— Reply to this email directly or view it on GitHub https://github.com/michaelcarter/mixpanel-data-export-js/issues/8#issuecomment-65555860 .

michaelcarter commented 9 years ago

@areagray. A long time coming but I've just published an update to pass through a csv string if you specify format: 'csv' formatting in your request parameters. This doesn't work in-browser at the moment because the script doesn't expect the text/csv content type, but should be fine in nodey environments.

Perhaps you could give it a try and let me know if it's what you'd expected? The release you're after is 1.6.2

areagray commented 9 years ago

thanks mike! will try to take it for a test run this week....

On Sun, Feb 15, 2015 at 5:21 AM, Mike Carter notifications@github.com wrote:

@areagray https://github.com/areagray. A long time coming but I've just published an update to pass through a csv string if you specify format: 'csv' formatting in your request parameters. This doesn't work in-browser at the moment because the script doesn't expect the text/csv content type, but should be fine in nodey environments.

Perhaps you could give it a try and let me know if it's what you'd expected?

— Reply to this email directly or view it on GitHub https://github.com/michaelcarter/mixpanel-data-export-js/issues/8#issuecomment-74417176 .

michaelcarter commented 9 years ago

@areagray Any thoughts on this? I'll close the issue this weekend and hope for the best unless I hear from you. :-)

areagray commented 9 years ago

Hi Mike!

I'm sorry I haven't tested it. I've been working with your lib all day, actually. I've been using it in meteor.js to do some processing with the people data. Interesting thing - unlike event data, which returns with no limit, engage data is paged. So, to get more than 1000 users back, i needed to create a version of a promises array to to manually page through the results, then return when all complete. Fun day.

Thanks again for creating this awesome utility.

Chris

On Wed, May 6, 2015 at 1:18 PM, Mike Carter notifications@github.com wrote:

@areagray https://github.com/areagray Any thoughts on this? I'll close the issue this weekend and hope for the best unless I hear from you. :-)

— Reply to this email directly or view it on GitHub https://github.com/michaelcarter/mixpanel-data-export-js/issues/8#issuecomment-99593525 .

areagray commented 9 years ago

Hi Michael,

I noticed I've started getting error messages back from Mix saying "Expired request" . Have you seen this recently?

Thanks for the insight, Chris G

On Sun, Jul 12, 2015 at 7:04 AM, Mike Carter notifications@github.com wrote:

Closed #8 https://github.com/michaelcarter/mixpanel-data-export-js/issues/8.

— Reply to this email directly or view it on GitHub https://github.com/michaelcarter/mixpanel-data-export-js/issues/8#event-353784429 .

areagray commented 9 years ago

disregard - figured out how to change the default timeout. cheers.

On Wed, Jul 29, 2015 at 11:48 AM, Christopher Gray areagray@gmail.com wrote:

Hi Michael,

I noticed I've started getting error messages back from Mix saying "Expired request" . Have you seen this recently?

Thanks for the insight, Chris G

On Sun, Jul 12, 2015 at 7:04 AM, Mike Carter notifications@github.com wrote:

Closed #8 https://github.com/michaelcarter/mixpanel-data-export-js/issues/8.

— Reply to this email directly or view it on GitHub https://github.com/michaelcarter/mixpanel-data-export-js/issues/8#event-353784429 .