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

Improve [Random] to use a custom mini browser #15

Merged
merged 5 commits into from
Jul 12, 2022

Conversation

Sarajiko
Copy link
Collaborator

@Sarajiko Sarajiko commented May 21, 2022

added the ability to filter by course via "course__slug" for requests against the mini browser

See ownrecipes/OwnRecipes#11.

@Sarajiko Sarajiko requested a review from sepulzera May 21, 2022 10:55
@Sarajiko Sarajiko force-pushed the feature/api/recipe-random-by-course branch from 76c1d0c to 6d2606c Compare May 21, 2022 11:02
@sepulzera
Copy link

Please add the new query_param to the OpenAPI document.

Quick instructions:

  1. Personally I use Stoplight Studio to edit OpenAPI documents. For Linux Mint, you can just download and run the AppImage.
  2. Get the latest api dev: git checkout development and git pull
  3. Make sure your user has write permission on the OpenAPI document: sudo chmod 666 docs/ownrecipes-api.json
  4. Run Stoplight Studio and open the OpenAPI document (at the start screen, use "Open Existing Folder" at the lower left corner).
  5. At the left sidepane, find the path "/recipe/mini-browse". Click on its GET.
  6. Add a [+ Query Param] for your changes.
  7. Save (ctrl + s) and do the git stuff.

@sepulzera sepulzera changed the title Improve [Random] to use a custom mini browser #11 Improve [Random] to use a custom mini browser (https:/ownrecipes/OwnRecipes/issues/11) May 26, 2022
@sepulzera sepulzera changed the title Improve [Random] to use a custom mini browser (https:/ownrecipes/OwnRecipes/issues/11) Improve [Random] to use a custom mini browser May 26, 2022
@sepulzera sepulzera added this to the v3.0.0 milestone Jun 27, 2022
…y to filter by course via "course__slug" for requests against the mini browser
@Sarajiko Sarajiko force-pushed the feature/api/recipe-random-by-course branch from 6d2606c to fb82092 Compare June 28, 2022 16:27
@sepulzera sepulzera merged commit 048691a into development Jul 12, 2022
@sepulzera sepulzera deleted the feature/api/recipe-random-by-course branch November 23, 2022 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants