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

Integrating RecipeClipper as a recipe parser #2267

Open
braccae opened this issue Apr 23, 2024 · 1 comment
Open

Integrating RecipeClipper as a recipe parser #2267

braccae opened this issue Apr 23, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@braccae
Copy link

braccae commented Apr 23, 2024

Is your feature request related to a problem? Please describe.
A huge limitation of the parser is that it doesn't work on most recipe sites. There's like 3 that I frequent that I'm able to pull with Nextcloud Cookbook. The rest I have to manually copy paste in.

Describe the solution you'd like
Recently I tried out Using RecipeSage and I noticed the parser they use is much more compatible and also a separate component from the main app, listed under RecipeClipper. I was wondering if it would be possible to integrate it as a supplementary parser for Nextcloud Cookbook.

@braccae braccae added the enhancement New feature or request label Apr 23, 2024
@christianlupus
Copy link
Collaborator

There was already a similar issue #388 where a switch to browser-sided parsing was made. In fact this has the same problem: As the web frontend we are not allowed to access foreign websites directly. So, this script cannot access the website unless we somehow include it in that page which is not as easy as it seems: a normal browser's job is to keep the web sites apart. (Would you like if Facebook would be able to parse all opened tabs and read your bank account data?)

It might be manageable by doing it using a bookmarklet #431 but needs further investigation.

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

No branches or pull requests

2 participants