forked from open-eats/OpenEats
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Improve [Random] to use a custom mini browser #11
Comments
The api has to be extended. More precisely, the MiniBrowseViewSet has to be extended to support course as filter. For the frontend there is quite more work to do. |
This improves the recipe footer so much! To make it even more awesome, we should also filter by cuisine, @Sarajiko . The frontend is prepared. :-) |
@sepulzera. I added a filter for cuisine__slug and one for tag__slug too, just in case. |
sepulzera
pushed a commit
to ownrecipes/ownrecipes-api
that referenced
this issue
Jul 12, 2022
This commit will extend the mini-browser endpoint. It will add filters for course(__slug) and cuisine(__slug). See: ownrecipes/OwnRecipes#11 Co-authored-by: rico <[email protected]>
sepulzera
added a commit
to ownrecipes/ownrecipes-web
that referenced
this issue
Jul 12, 2022
sepulzera
added a commit
to ownrecipes/ownrecipes-web
that referenced
this issue
Jul 12, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The [Random] recipe function is not much useful for us. Usually you are searching for a course. If you are looking for a main course, you probably have to hit the [Random] multiple times to skip appetizers, desserts and such.
It would be way more convenient to go to a randomized mini browser page. There you would select the course (default: Main Course) and get displayed 6 randomized recipes for this course.
Maybe that could be enhanced later, f. e. to include another option to filter only seasonal recipes.
The text was updated successfully, but these errors were encountered: