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

scraper not getting food network info #1119

Open
AnnieSixgun opened this issue May 4, 2024 · 3 comments
Open

scraper not getting food network info #1119

AnnieSixgun opened this issue May 4, 2024 · 3 comments
Labels

Comments

@AnnieSixgun
Copy link

AnnieSixgun commented May 4, 2024

The URL of the recipe(s) that are not being scraped correctly

- https:// ...
https://www.foodnetwork.com/recipes/ina-garten/asian-grilled-salmon-recipe-1944413
...

the recipe info

...

Looks Like We Couldn't Find Anything

...

@AnnieSixgun AnnieSixgun added the bug label May 4, 2024
@jknndy
Copy link
Collaborator

jknndy commented May 4, 2024

@AnnieSixgun , I could not replicate the issue here. Could you double check that you're using the latest version of the library?

>>> from recipe_scrapers import scrape_me
>>> scraper = scrape_me('https://www.foodnetwork.com/recipes/ina-garten/asian-grilled-salmon-recipe-1944413')
>>> scraper.to_json()
{'author': 'Ina Garten Bio & Top Recipes', 'canonical_url': 'https://www.foodnetwork.com/recipes/ina-garten/asian-grilled-salmon-recipe-1944413', 'category': 'main-dish', 'cook_time': 9, 'cuisine': 'asian', 'description': "Ina Garten's Asian Grilled Salmon, from Barefoot Contessa on Food Network, is a light, quick main dish with salty-sharp flavors like Dijon and soy sauce.", .... }
>>> 

@mlduff
Copy link
Contributor

mlduff commented May 5, 2024

@AnnieSixgun I also couldn't replicate the issue, although I did notice that prep_time and cook_time have not been implemented despite them being available in the schema - @jknndy did you want me to make this improvement?

@jknndy
Copy link
Collaborator

jknndy commented May 5, 2024

@AnnieSixgun I also couldn't replicate the issue, although I did notice that prep_time and cook_time have not been implemented despite them being available in the schema - @jknndy did you want me to make this improvement?

Up to you @mlduff ! I've been exploring this issue on a larger scale to see if we can populate all possible coverage that is available so it'll get caught in that regardless

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants