serpapi / public-roadmap

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

[Google Search API] Company `subsidiaries` are mixed up #1654

Closed ocvit closed 2 weeks ago

ocvit commented 3 weeks ago

When company subsidiaries are grabbed from both - carousel and knowledge graph - subsidiaries array includes strings and hashes at the same time. Also, in case of Apple subsidiaries query, More string should be ignored.

Screenshot 2024-06-05 at 13 59 38

Playground | Inspect

ocvit commented 3 weeks ago

Consecutive requests using the same query returned different subsidiaries in a knowledge graph - with MORE and 51 more at the end. In these cases, problem is not present with a correct array of hashes. Looks like something is wrong with More specifically.

ocvit commented 3 weeks ago

Affected HTML and JSON

zyc9012 commented 3 weeks ago

Affected HTML and JSON

Let's just provide the inspector link in the first message.

e.g.

Playground Inspect

ocvit commented 2 weeks ago

The fix has been released.

subsidiaries field (and other facts with · <more suffix>) in conjunction with the top carousel is parsed correctly now.

Screenshot 2024-06-10 at 19 20 01