Skip to content
This repository has been archived by the owner on Jun 16, 2021. It is now read-only.

All subtask completed #8

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

aishwary023
Copy link
Member

@aishwary023 aishwary023 commented May 16, 2020

CSoC Task 3 Submission

I have completed the following tasks

  • Basic endpoints
  • Collaborator feature

@nishantwrp
Copy link
Member

Hi @aishwary023,
Nice work! Here are some suggestions.

  • Adding a collab with user that doesn't exist gives 500 error., and that should never happen.
  • PUT or PATCH doesn't work fine in swagger view.
  • There is no differentiation between creator and collab.
  • You could've used custom permissions.py for managing user permissions.

Your score has been updated.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants