zroberts / walmart

GNU General Public License v3.0
0 stars 0 forks source link

Project #1

Closed lcox2 closed 7 years ago

lcox2 commented 7 years ago

Trying to get your project to work in our searching function, but for some reason it doesn't want to acknowledge the existence of your methods, etc. I have followed the steps in the readme to a T. But for some reason it says your 'Resp[i]' Array isn't defined.

This is bizarre, to say the least.

zroberts commented 7 years ago

Can you paste in your code for the Walmart call-in?

Thanks ~Zak

lcox2 commented 7 years ago

var walmartObject = { searchTerm: request.body.name, minPrice: '0', maxPrice: '10000000' } walmart.search(walmartObject, function(err, res){

});

});

I pushed it back to before I made the commits, considering they didn't work. I just need to know what goes below. Request.body.name is where we take the name from the request that is sent. That part works, it takes the search, there is just no response.

zroberts commented 7 years ago

walmart.search(walmartObject, function(err, res){

console.log(res); });

Try that, should return an array of objects.

From there you'll have to loop through that array and manipulate the response on your end.

lcox2 commented 7 years ago

Gotcha

lcox2 commented 7 years ago

And to reply to your questions about our Repo. I am concerned as to why its not working. But I think its a consistancy issue. Something about the Master Branch not being up to date with the testing branch. Thats what Aaron explained it as

I told John to approve the Merge request, but for some reason he hasn't.

lcox2 commented 7 years ago

And it keeps sending undefined to the console. When I log the response, its undefined.

zroberts commented 7 years ago

Have you tried If(err){ Console.log(err); }

To see if you are getting any error instead of the response?

lcox2 commented 7 years ago

I get an error saying "cannot log error of undefined"

On Dec 6, 2016 7:05 AM, "Zachary Roberts" notifications@github.com wrote:

Have you tried If(err){ Console.log(err); }

To see if you are getting any error instead of the response?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/zroberts/walmart/issues/1#issuecomment-265133992, or mute the thread https://github.com/notifications/unsubscribe-auth/AUQ88YEij4Ei_ZdWDh5hEHgpVg3WtslYks5rFU98gaJpZM4LE6WN .

zroberts commented 7 years ago

Post in your code again please

zroberts commented 7 years ago

I updated the readme with a full example, see if that helps clear things up, it's at the bottom, let me know if you're still having issues.

Thanks ~Zak

lcox2 commented 7 years ago

Its all good Zach

Yours is working fine, but matts is having some issues with dotnev. Or at least I am.

Did you get ours to work, or are you still having issues?

zroberts commented 7 years ago

I haven't been able to get yours to work.

Matt changed it so you don't need the dotenv, he must not have updated the readme. just pass the ebay api key as the first parameter in the ebay call, like this.

ebay(ebayKey,queryObject.searchTerm, ebayMaxPrice, function(err, res){ if(err){ console.log(err); else{ console.log(res); } }