simplyspoke / node-harvest

Node.js Client API for the Harvest time tracking service
http://simplyspoke.github.io/node-harvest/
MIT License
106 stars 58 forks source link

Can't use slim and of_user in TimeTracking.daily #56

Closed jayjanssen closed 7 years ago

jayjanssen commented 7 years ago

https://github.com/log0ymxm/node-harvest/blob/master/lib/time-tracking.js#L17-L27

Notice that setting 'slim' appends a '?' to the URL irrespective of whether of_user has been set or not.

DerekDomino commented 7 years ago

You may look at the recently merged PR that use append-query to handle the query string: https://github.com/log0ymxm/node-harvest/pull/55/files

simplyspoke commented 7 years ago

Closing as suggestion was already merged.