New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: newegg realtime api #664
Conversation
I don't program in node/javascript anymore, and generally write code in Rust or Python... so not overly familiar with nodejs Whatever you need me to fix let me know. |
Join the club 😆 I'll go over this later today. Thanks for the hard work on this one! |
…m/jbwilliams1/nvidia-snatcher into develop-newegg-realtime-api-fetch
Sorry for all of the commits, I was annoyed by the linter not passing :)
|
No problem. I appreciate you sticking through it 😀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is awesome! I appreciate you putting in this work. Let's see how it goes!
Signed-off-by: Jef LeCompte <jeffreylec@gmail.com>
I went ahead and fixed up some dependency conflicts, hope you don't mind. Let's get this in here! |
Description
Added realtime inventory lookup for Newegg cards
Resolves #646
Resolves #643
Testing
Used an in-stock card for the test series, validated it worked on an in-stock and out of stock card. Runs normally on out-of-stock cards
New dependencies
None (well, node-fetch, but it's bundled in)