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.

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

Add support for .pdf response #12

Open
JulianJorgensen opened this issue Sep 5, 2017 · 3 comments
Open

Add support for .pdf response #12

JulianJorgensen opened this issue Sep 5, 2017 · 3 comments

Comments

@JulianJorgensen
Copy link

JulianJorgensen commented Sep 5, 2017

Great work on this library! Big thanks @estliberitas !

Wanted to request support for .pdf response. According to their docs you just need to add .pdf to the call URL: https://github.com/toggl/toggl_api_docs/blob/master/reports.md

e.g.
GET https://toggl.com/reports/api/v2/summary.pdf

Cheers

@JulianJorgensen
Copy link
Author

JulianJorgensen commented Sep 6, 2017

@estliberitas I've forked this repo and added ".pdf" to the Toggl api calls. Not sure if I need to also modify some of the other code to support file streaming?

Just spent the day trying to fetch the pure PDF, but with no luck.

P.s. I've also asked StackOverflow for any pointers: https://stackoverflow.com/questions/46070781/get-toggl-pdf-report-using-node

@estliberitas
Copy link
Member

@JulianJorgensen Hey man. Well, I can look into it during weekend, or you can make a PR, hehe. I wonder if you can do it with a usual Node http client.

@JulianJorgensen
Copy link
Author

JulianJorgensen commented Oct 2, 2017 via email

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

No branches or pull requests

2 participants