Skip to content

Fix the error that the returned result is missing the name field - #15

Open
jelly-hst wants to merge 1 commit into
amineboutarfi:masterfrom
jelly-hst:fix_lost_name
Open

jelly-hst wants to merge 1 commit into
amineboutarfi:masterfrom
jelly-hst:fix_lost_name

Conversation

@jelly-hst

@jelly-hst jelly-hst commented Feb 20, 2024

Copy link
Copy Markdown

Hi @amineboutarfi, Your code is great, however I got this error when using it.
When using

python main.py

or

python main.py -s="dentist boston" -t=n

If n > total count then the returned result will be missing the name field.
Specifically, I just added one line of code like this

@amineboutarfi

Copy link
Copy Markdown
Owner

@hosythach I believe the Blank name issue was caused by Google changing the HTML, hence the xPath needed to change. Please confirm that it is now working?

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