-
Notifications
You must be signed in to change notification settings - Fork 44
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
Comments
@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 |
@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. |
Ah yea that may work!
As a temporary solution I just ended up embedding the “Saved Report” as a iframe instead.
Cheers
… On Sep 19, 2017, at 4:46 AM, Alexander Makarenko ***@***.***> wrote:
@JulianJorgensen <https://github.com/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.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#12 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAC1C3TQ3Nr7GeU19R-crukz3Ut5cE2Qks5sjtapgaJpZM4PMiuA>.
|
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
The text was updated successfully, but these errors were encountered: