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.

Sign up for GitHub

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

Jump to bottom

Add seller parameters #12

Merged
merged 1 commit into from
Oct 5, 2020
Merged

Add seller parameters #12

merged 1 commit into from
Oct 5, 2020
+19 −4

Conversation

rpdswtk
Copy link
Contributor

@rpdswtk rpdswtk commented Oct 4, 2020 •
edited
Loading

Issue: #1

Changes:

  • Added seller property to user database model
  • Modified /register resource to handle seller option
  • Modified /store POST and DELETE methods to check if user is a seller. 403 status is returned if the user is not a seller.

Comments:

  • Not sure how to approach github registration regarding this feature

@DiptoChakrabarty
Copy link
Owner

Thanks for the contributions , merged .
If you like the project please do star it , I will be working on this more and also might put up further issues that I wish to implement if you want to check those out.

@DiptoChakrabarty DiptoChakrabarty merged commit ed1a8a5 into DiptoChakrabarty:development Oct 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DiptoChakrabarty DiptoChakrabarty DiptoChakrabarty approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rpdswtk @DiptoChakrabarty