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

It would max at 2 or 3 reviews and stop #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kyle10n
Copy link

@kyle10n kyle10n commented Mar 6, 2019

removing the ::2 index allows the scrapper to find the 5 reviews then go to the next page.

If you don't the script thinks it has found all reviews on the first page then stops.

I think tripadvisor updated their website so this happened.

removing the ::2 index allows the scrapper to find the 5 reviews then go to the next page.

If you don't the script thinks it has found all reviews on the first page then stops.
@fuad47
Copy link

fuad47 commented Dec 30, 2019

You are right. I suggest replacnig class name for num_reviews to
''hotels-hotel-review-about-with-photos-Reviews__seeAllReviews--3PpLR"
and adding this line:
num_reviews = num_reviews.replace('review', '')

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.

2 participants