Closed heathrampazis closed 2 months ago
Sweet thanks for the suggestions and review @mlduff. I did attempt to add the category functionality when I was writing the scraper but was having issues. Your way of doing it seems a lot more intuitive, so I will implement this now!
Yep, thanks @mlduff - in terms of the category info: Bakery
and Patisserie
do seem like worthwhile categories from the recipe; is Finished Product
a category though? I'm not sure it is and would probably omit it.
@jayaddison - I have implemented the Category changes in my latest commit. I don't think the recipe schema supports Finished Product, so I haven't touched on that as I didn't think it was necessary. Let me know what you think.
Ohhh, now I understand; Finished Product
is not a category itself, but it's the heading used to describe each recipe. So this one is for glaze, and others are for donuts / pastries, etc. Ok, hm. I think it's still OK to omit that, but there is a chance it could be useful to some people. I don't mind either way, so I'll continue reviewing the rest of the code.
Let me know if-and-when you're ready for this to be merged and I'll go ahead and do that.
@jayaddison - I'm happy for it to be merged 🔥
This pull request adds support for Bakels Australia recipes.
There was some schema support for this website. However, HTML scraping with soup needed to be used to correctly retrieve the ingredients, instructions, and description.
Resolves #1096