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

Peer reviewers #9

Open
wants to merge 28 commits into
base: main
Choose a base branch
from
Open

Peer reviewers #9

wants to merge 28 commits into from

Conversation

sgfost
Copy link
Owner

@sgfost sgfost commented Oct 29, 2024

No description provided.

sgfost and others added 28 commits September 30, 2024 11:22
- use get_or_create to update a reviewer when trying to create
  while one already exists

- allow users to create or update their own reviewer record. this allows
  anyone to insert themselves into the reviewer pool at any time, should
  consider whether this is desirable or we should add a pending status

Co-authored-by: Karthik Bandagonda <[email protected]>
This prevents the reviewer object on the parent component from being mutated before the form is submitted
permission class was not set up properly, now correctly drops through
to has_object_permission for retrieve, update, etc.

* add a profile link to reviewer card
provide string representation of PeerReviewer
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.

3 participants