hhursev / recipe-scrapers

Python package for scraping recipes data
MIT License
1.61k stars 505 forks source link

thecookingguy missing fields #1070

Closed aecai closed 2 months ago

aecai commented 2 months ago

Resolves #3, #4, #5, and #6.

Overview:

Returns None for mandatory total_time field since recipe from thecookingguy.com does not provide it and removes missing optional fields from file. Changes: 'recipe_scrapers/thecookingguy.py'

Returns None for mandatory missing field Deleted optional field sections

Testing: Ran the following commands in the python console and got the expected output.

from recipe_scrapers import scrape_me
scraper = scrape_me("https://www.thecookingguy.com/recipes/chili-cumin-lamb")
scraper.ingredients()

Note: optional fields should be excluded from json test as per https://github.com/hhursev/recipe-scrapers/issues/957

jknndy commented 2 months ago

Hi @aecai , was this meant to be a pull request at @rmdluo 's repo instead of the main repo? Once merged there it can be updated on the original pull request