pat310 / google-trends-api

An API layer on top of google trends
https://www.npmjs.com/package/google-trends-api
MIT License
889 stars 176 forks source link

Trending Searches API additions #117

Closed Colibry-Source closed 5 years ago

Colibry-Source commented 5 years ago

Hi this my tested proposal for Trending Searches api which reuses your framework .

coveralls commented 5 years ago

Coverage Status

Coverage increased (+0.4%) to 98.148% when pulling 3f035cc38ec7bb0f060739ec34614a9765c0d5d3 on Colibry-Source:master into 58d97b5edbdc3cc883b2d89b61edc45175696a41 on pat310:master.

pat310 commented 5 years ago

Looks good! @Colibry-Source

pat310 commented 5 years ago

You may need to pull master, I merged a different PR earlier

Colibry-Source commented 5 years ago

You may need to pull master, I merged a different PR earlier Yeah I def will need to do this.

Oh boy... gotta figure out how to do this effectively ... looks like a whole bunch of cutting and pasting is about to happen.

;)

Going to read-up about this.

Colibry-Source commented 5 years ago

Hi.

I have updated my source and incorporated the changes from v. 4.8.

Please review at your leisure.

Pat @ Colibry

Colibry-Source commented 5 years ago

Closing this pull request as it was for an older branch of master and trying to request once more.