Skip to content
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

Added price #19

Merged
merged 2 commits into from
Mar 6, 2019
Merged

Added price #19

merged 2 commits into from
Mar 6, 2019

Conversation

bitofbreeze
Copy link
Contributor

I found a relatively lightweight way to retrieve pricing info the way some of the price-tracking applications do it :D.
It uses a special path gp/cart/desktop/ajax-mini-detail.html/ref=added_item_1?ie=UTF8&asin=
It works on everything I’ve found except e-books.
It is an extra resource about 3KB to load, so maybe we should make it an optional parameter but it can also be used to get number of ratings, if in stock, seller, etc. (e.g., https://www.amazon.com/gp/cart/desktop/ajax-mini-detail.html/ref=added_item_1?ie=UTF8&asin=1593276036)

I'm pretty confident about this method, and it might even serve us well to get all the info this way since I've noticed images aren't always correct as is.

@bitofbreeze bitofbreeze mentioned this pull request Jan 18, 2019
@coveralls
Copy link

Pull Request Test Coverage Report for Build 204

  • 6 of 6 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 98.857%

Totals Coverage Status
Change from base Build 199: 0.04%
Covered Lines: 173
Relevant Lines: 175

💛 - Coveralls

@tducret
Copy link
Owner

tducret commented Feb 2, 2019

Hi @CSFlorin . Very small change (nice!).
Any thought about adding it to the CSV output?

@tducret
Copy link
Owner

tducret commented Feb 2, 2019

I was implementing the CSV part, but I am encountering captcha when requesting 100 items :(
Do you have similar behaviour @CSFlorin ?

@bitofbreeze
Copy link
Contributor Author

i think so :( not sure how amazon fingerprints: may just need to rotate the user agent or may have to use proxies...

@tducret tducret merged commit d0cd60a into tducret:master Mar 6, 2019
@jpeacock29
Copy link
Contributor

Does this close #5?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants