serpapi / public-roadmap

Public Roadmap for SerpApi, LLC (https://serpapi.com)
45 stars 3 forks source link

[Google Search API] Order pickup/Order delivery link is parsed as `website` in the `knowledge_graph` #1575

Closed marm123 closed 2 weeks ago

marm123 commented 2 months ago

One of our high-volume customers reported that we're parsing the wrong link in the website endpoint. It should be a direct link to the business website, but instead, we're parsing the Order pickup/Order delivery Google Food link. I'm marking it as urgent, as it's affecting their business.

Playground example:

image

What we're parsing in the website field:

image

Where the direct Website link leads to:

image

A bit different but similar issue to: https://github.com/serpapi/public-roadmap/issues/1557

Playground | Inspect | Google search Intercom

btaunt commented 2 months ago

This has been updated.

We now parse the correct website URL on both desktop and mobile, along with targeting the"actionable" buttons for Google's newer layout (ie. order_pickup etc.)


Screenshot 2024-05-02 at 9 10 37 AM Screenshot 2024-05-02 at 9 11 18 AM Screenshot 2024-05-02 at 9 11 28 AM

Note: the desktop website URL is still awaiting a small update in a separate PR. I will update this ticket again once that is officially merged.

schaferyan commented 2 months ago

Thanks @btaunt! The customer noted that it looks like the website url is still not being parsed correctly for the original Desktop example:

https://serpapi.com/playground?q=indian+tanpura+new+york+ny+10024&location=Austin%2C+Texas%2C+United+States&gl=us&hl=en&device=desktop&no_cache=true&google_domain=google.com&newPara=votes+site+chips+stick+highlight+lr+as_qdr+safe+google_domain+si+form+filter+as_epq+client

Will that be fixed in the update you mentioned?

I'm re-opening this for now as from the customer's perspective it hasn't been fixed yet.

btaunt commented 2 months ago

Will that be fixed in the update you mentioned?

That's correct @schaferyan - sorry for not being more clear! I will update here once it is live on production 👍

schaferyan commented 2 months ago

No worries, thanks @btaunt!

btaunt commented 2 months ago

The fix is now live. cc. @schaferyan

Example searches:

schaferyan commented 2 months ago

Thanks @btaunt !

marm123 commented 2 weeks ago

I'm reopening this, as it looks like the issue is still there from time to time. Unfortunately, I wasn't able to recreate it, as it's not consistent, but the customer provided a couple of inspect link, where the issue is present:

Inspect Inspect Inspect Front

btaunt commented 2 weeks ago

We have specifically targeted those edge cases (based on the search inspections provided to us).

The proper website will be parsed for those layouts, if they happen to appear again.