Issue with two "encodeURIComponent" instead of one

[KirianCaumes]

Hi,
Just noticed an issue in the last version ("2.8.5") with the method "findItemsByKeywords".
You are calling two times "encodeURIComponent" on the keywords variable :

//findingApi.js l.18
//...
options.keywords = encodeURIComponent(options.keywords);
//...

then

//common-utils/index.js l.22
//...
else if (key === 'keywords' || key === 'categoryId' || key === 'productId' || key === 'sortOrder' || key === 'storeName') {
    const encodeParam = encodeURIComponent(options[key]);
    params = `${params}${key}=${encodeParam}&`;
}
//...

So keywords like "foo bar" would look like "foo%2520bar" instead of "foo%20bar", resulting in a wrong search against Ebay's API.

Noticed that this issue might be present in other piece of codes, but I haven't checked it.

Hope this will help 😉

[pajaydev]

@KirianCaumes Thanks for pointing this out. I appreciate it 😄 . Will fix it and release a new version.

[pajaydev]

@KirianCaumes I have fixed it. Kindly use this version
npm install ebay-node-api@2.8.6

[KirianCaumes]

No problem 😁
It works fine now, thanks !