search

hhursev / recipe-scrapers

Python package for scraping recipes data
MIT License
1.7k stars 521 forks source link

fix(mob): updating to align with new website structure #977

Closed jacksgreen closed 8 months ago

jacksgreen commented 9 months ago

Fixes #972

jayaddison commented 8 months ago

@jacksgreen could you update the description of this to include the text 'Fixes #977'? (that'll mean that your bugreport will be automagically closed if+when this is merged).

jacksgreen commented 8 months ago

Fixes #977

Sure, done 👍

jayaddison commented 8 months ago

Ah, oops: I think that was the PR title you updated, instead of the description field (the title is recommended to be fairly short, and the description is for longer-form information describing the pull request).

jacksgreen commented 8 months ago

Ah, oops: I think that was the PR title you updated, instead of the description field (the title is recommended to be fairly short, and the description is for longer-form information describing the pull request).

Got you! Added now

jacksgreen commented 8 months ago

@jayaddison @CrossR I've updated the tests so they pass now, let me know if this can be merged!

jayaddison commented 8 months ago

Looks good, merging in a few moments - thanks @jacksgreen! This will be included in an upcoming version of recipe-scrapers soon.

I'm going to fix one more detail in the description - I recommended linking this to #977, but that's this pull request (!) - instead what I should have written #972, the issue that this resolves/fixes.