dotnet / fabricbot-config

Scripts for managing fabricbot configurations in the dotnet repos
MIT License
4 stars 7 forks source link

Only resurrect issue cards if the comment author has (at most) Read permission #24

Closed jeffhandley closed 2 years ago

jeffhandley commented 2 years ago

Fixes #6 by inverting the permission logic. It turns out that the condition that checks the user's permission is an "at most" check instead of an "at least" check. Therefore, checking for "write" permission doesn't work if the user has admin or maintain permissions (or triage permissions).

With this change, we will only resurrect issue cards (for needing further triage) if the comment author has (at most) Read permission. This behavior was tested via #23.