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

added action editor view #1116

Open
wants to merge 6 commits into
base: action_editor_db
Choose a base branch
from
Open

Conversation

shreyalpandit
Copy link
Collaborator

Added a view for users who are an action editor for a particular event. They are able to view the responses that they were assigned to and assign reviewers to a response.
(Still need to add tests for action editor permissions)

image

image

@shreyalpandit shreyalpandit requested a review from avishkar58 April 3, 2023 13:39
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.

This is really well done, thank you Shreya!!

if current_user.is_event_admin(event_id):
responses = response_repository.get_all_for_event(event_id, not include_unsubmitted)

elif current_user.is_action_editor(event_id):
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if there's a better name than "action-editor" that's not a journal-specific term, since the functionality is not limited to a journal event type. Will try think of something too!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I agree! I was thinking something along the lines of evaluator, critic, etc. What do you think?

api/app/responses/api.py Outdated Show resolved Hide resolved
api/app/responses/api.py Outdated Show resolved Hide resolved
api/app/reviews/api.py Outdated Show resolved Hide resolved
api/app/utils/auth.py Show resolved Hide resolved
webapp/src/pages/ResponsePage/ResponsePage.js Outdated Show resolved Hide resolved
@avishkar58
Copy link
Contributor

Will approve once you've added the tests for action editor permissions :)

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