Skip to content

Add remove failing invites step #55

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

Merged
merged 2 commits into from
Oct 9, 2024
Merged

Conversation

cunla
Copy link
Member

@cunla cunla commented Oct 4, 2024

No description provided.

Copy link
Member

@Stormheg Stormheg left a comment

Choose a reason for hiding this comment

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

I don't think this step should be part of the new member playbook. To me, this is something separate because it has no direct relation to the task at hand: inviting a new member.

I'd suggest we create a 'scheduled admin task' section and do this step every 3 months or so.

@cunla
Copy link
Member Author

cunla commented Oct 6, 2024

We can discuss it again in the next meeting.

The main points raised about it in the last meeting:

  • We want the terraform files to reflect the org-state => we should remove failed-invites.
  • At the moment, we do not know how much of an issue it is, and therefore we should not invest in automating this process for now.
  • Having a scheduled task would mean additional admin work (assigning someone, etc.)
  • As a balanced solution, we agreed for now it can be a part of adding members.

I do think if this becomes a bigger problem, it would make sense to invest in automating this process and having it run periodically.

Copy link
Member

@tim-schilling tim-schilling left a comment

Choose a reason for hiding this comment

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

I had the same reservation as Storm, but if that's what the other three decided in the meeting, I'm good with this change. I can get on board with trying to add it to an existing process.

@tim-schilling tim-schilling force-pushed the add-failing-invites-step branch from e161880 to 3d7e5c5 Compare October 9, 2024 01:48
@tim-schilling tim-schilling merged commit ccd1527 into main Oct 9, 2024
@tim-schilling tim-schilling deleted the add-failing-invites-step branch October 9, 2024 01:48
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.

3 participants