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

Action Editor DB #1115

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Action Editor DB #1115

wants to merge 4 commits into from

Conversation

shreyalpandit
Copy link
Collaborator

Creates new database to track which action editor is assigned to a response.

Copy link
Contributor

@avishkar58 avishkar58 left a comment

Choose a reason for hiding this comment

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

Hi Shreya, can you remind me what exactly an action-editor is meant to be please?

@avishkar58
Copy link
Contributor

Hi Shreya, can you remind me what exactly an action-editor is meant to be please?

Oh nevermind, it makes sense now after seeing your next PR :) Some kind of admin that is assigned to individual responses.

@shreyalpandit
Copy link
Collaborator Author

Yes! There are three main roles: Event Admin, Action Editor, and Reviewer. The Event Admins are tracking the submission of the submissions but aren't actually reviewing or approving. They will assign the action editor to a submission and this person is responsible for adding reviewers and setting the outcome of the submission. Action editors can be reviewers for submissions that they are not assigned to as an action editor role, but reviewers cannot be action editors.

Copy link
Contributor

@avishkar58 avishkar58 left a comment

Choose a reason for hiding this comment

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

Awesome, thanks for the explanation!

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