simplyspoke / node-harvest

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

fix: correctly pass query args to GET endpoints #536

Closed ThisIsMissEm closed 3 years ago

ThisIsMissEm commented 3 years ago

Previously the query args for the GET endpoints weren't correctly passed, this fixes that issue.

Fixes: https://github.com/simplyspoke/node-harvest/issues/219

ThisIsMissEm commented 3 years ago

Thanks for merging this @simplyspoke! Is there additional work required before this can be released?

simplyspoke commented 3 years ago

Looks like I will need to check the automation. Hopefully later today, at least this week for sure.

simplyspoke commented 3 years ago

:tada: This PR is included in version 2.2.6 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

simplyspoke commented 3 years ago

@ThisIsMissEm It should be published if you want to pull it down and ensure its working. CI was out of alignment, so had to running things locally to sort it.