goferito / node-bing-api

Node.js module for the Bing Search API (Cognitive Services)
MIT License
56 stars 28 forks source link

searchVertical does not respect offset option. #33

Closed darionco closed 7 years ago

darionco commented 7 years ago

Sorry for the quick report.

file: bing.js line: 69 function: searchVertical

is:

+ (opts.offset ? "&offset=" + opts.skip : "")

should be either:

+ (opts.offset ? "&offset=" + opts.offset : "")

or:

+ (opts.skip ? "&offset=" + opts.skip : "")
darionco commented 7 years ago

oh I just saw #29

goferito commented 7 years ago

Yup, it's fixed on v3.2.1, published an hour ago :smile: Thanks for the report anyway. Don't forget to :star: the repo :smile: