search

ecologylab / BigSemanticsWrapperRepository

Repository of wrappers used by the BigSemantics project.
Apache License 2.0
3 stars 9 forks source link

Add newegg_list #21

Closed zyue1105 closed 10 years ago

zyue1105 commented 10 years ago

add newegg_list that is pretty much like amazon_list to extract pages like http://www.newegg.com/All-Laptops-Notebooks/SubCategory/ID-32?Pagesize=100

quyin commented 10 years ago

Good work!

My main concern is that I thought you should have put some fields inside the items collection to extract more details. Did you see how that will work? Or did you have any problems of doing that?

There are also some minor style issues, but I'll merge this pull request into master and fix some of the issues myself, since we need it. Please see inline comments.