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

[DRAFT] Proposal reviewing mechanisms #34

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

sheenarbw
Copy link
Collaborator

This is a work in progress. If anyone wants to poke around please go ahead.

Basically how it works is:

  • Users can be marked as reviewers with the is_reviewer flag
  • Reviewers get access to a review dashboard where they can see proposals that need to be reviewed. It would also be nice to show some stats there, like how many things still need to be reviewed. It would also be good to do some filtering and sorting
  • the ReviewAspect model is used to say how reviews are done. You can set up different scoring mechanisms. Eg rate the Title out of 5, rate the relevance out of 3, etc
  • When a Reviewer reviews a talk, we generate a form based on the ReviewAspects that were set up
  • People should be able to edit their reviews as well

This PR will not include functionality around making final selections. It's just for the reviewers

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line 8, you forgot the rest two " on the docstring 🙂

Copy link
Collaborator

@theShinigami theShinigami Dec 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When creating a new proposal on create_proposal view, the forms module is getting overridden by django's forms it's not using .forms 🙂

image

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

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.

2 participants