Skip to content

Commit

Permalink
Fix unclaim actions not including the user ID
Browse files Browse the repository at this point in the history
  • Loading branch information
TimJentzsch committed Oct 23, 2022
1 parent 3ec24d4 commit 19c7857
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions blossom/api/slack/messages/unclaim.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ def get_ask_confirmation_blocks(
text_block,
_get_submission_info_block(submission, user),
{"type": "divider"},
_get_ask_confirmation_buttons_block(submission),
_get_ask_confirmation_buttons_block(submission, user),
]


Expand Down Expand Up @@ -86,7 +86,9 @@ def _get_submission_info_block(submission: Submission, user: BlossomUser) -> Dic
}


def _get_ask_confirmation_buttons_block(submission: Submission) -> Dict:
def _get_ask_confirmation_buttons_block(
submission: Submission, user: BlossomUser
) -> Dict:
"""Get the action buttons for the confirmation of an unclaim request."""
return {
"type": "actions",
Expand All @@ -95,12 +97,12 @@ def _get_ask_confirmation_buttons_block(submission: Submission) -> Dict:
"type": "button",
"style": "primary",
"text": {"type": "plain_text", "text": "Confirm"},
"value": f"unclaim_confirm_{submission.id}",
"value": f"unclaim_confirm_{submission.id}_{user.id}",
},
{
"type": "button",
"text": {"type": "plain_text", "text": "Cancel"},
"value": f"unclaim_cancel_{submission.id}",
"value": f"unclaim_cancel_{submission.id}_{user.id}",
},
],
}

0 comments on commit 19c7857

Please sign in to comment.