Denhac / denhac-webhooks

This repo holds some of our membership automation stuff. It listens to webhooks from the main wordpress site
7 stars 2 forks source link

Better handling of failed GitHub invites #56

Closed Jnesselr closed 5 months ago

Jnesselr commented 5 months ago

I realized we could grab failed invitations, but that wasn't on the team level, it was only on the organization level. Given how we're inviting people to the team and it also invites them to the denhac GitHub organization, I decided this change would be part refactor of the API layer and part fixing an issue.

Before these changes, the issue checker would report an error for anyone who is a member, but their GitHub invitation expired before they accepted the invitation. This PR adds a command that will automatically clear these member's GitHub username in their profile as well as send them an email letting them know their invite expired and how to get a new invite.

Currently the issue checker in the webhooks project has 74 issues, 36 of which are affected by this issue.