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

Add Wishlist Feature #3433

Closed
wants to merge 5 commits into from
Closed

Conversation

Chamarsh
Copy link

@Chamarsh Chamarsh commented Dec 9, 2024

Description

This pull request implements a wishlist feature. It allows users to add recipes to the wishlist.

Changes

  • Add Wishlist Model, Serializer, Viewset, Route
  • Add Model template, vue page
  • Add Testing for routes

Fixes #3377

Note

This PR is for a school project and is not 100% functional. Not to be merged.

@CLAassistant
Copy link

CLAassistant commented Dec 9, 2024

CLA assistant check
All committers have signed the CLA.

@vabene1111
Copy link
Collaborator

as discussed on discord (because of #2976 ) and especially since its not 100% functional closing this.

Please tell your teachers that stuff like this does not help open source if its not meant to be a real contribution because it wastes my time responding and looking at these kinds of things.

Still from a code perspective and even tests this looks pretty good.

@vabene1111 vabene1111 closed this Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement a wishlist feature
4 participants