Skip to content

Prevent invalid user management actions #425

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Prevent invalid user management actions #425

wants to merge 1 commit into from

Conversation

LanesGood
Copy link
Member

This PR:

  • Fixes prevent of self-removal from a team (type coercion error)
  • Requires more than 1 moderator to demote own moderator status

@LanesGood LanesGood requested a review from kamicut March 14, 2023 20:54
@vercel
Copy link

vercel bot commented Mar 14, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
osm-teams ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 14, 2023 at 8:55PM (UTC)

This was linked to issues Mar 14, 2023
Forbid last team moderator from downgrading own moderator status #313
Closed
Forbid only team moderator from removing self from team #314
Closed
@LanesGood
Copy link
Member Author

Superseded by #415 - including the same changes

@LanesGood LanesGood closed this Mar 16, 2023
@LanesGood LanesGood deleted the prevent-invalid-user-actions branch March 16, 2023 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kamicut kamicut Awaiting requested review from kamicut

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@LanesGood