Skip to content
Snippets Groups Projects
Commit 4a72b91c authored by Fredrik Jonsson's avatar Fredrik Jonsson
Browse files

Bugfix for notification to second reviewer.

parent d8c2a33a
No related branches found
No related tags found
No related merge requests found
...@@ -399,7 +399,7 @@ class SlackAdapter(AdapterBase): ...@@ -399,7 +399,7 @@ class SlackAdapter(AdapterBase):
# Notify second reviewer when first reviewer is done. # Notify second reviewer when first reviewer is done.
if message_type == MESSAGES.NEW_REVIEW and related: if message_type == MESSAGES.NEW_REVIEW and related:
if submission.assigned.with_roles().count() == 2 and related.author == submission.assigned.with_roles().first().reviewer: if submission.assigned.with_roles().count() == 2 and related.author.reviewer == submission.assigned.with_roles().first().reviewer:
recipients.append(self.slack_id(submission.assigned.with_roles().last().reviewer)) recipients.append(self.slack_id(submission.assigned.with_roles().last().reviewer))
return recipients return recipients
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment