Skip to content
This repository has been archived by the owner on Mar 29, 2022. It is now read-only.

User management #464

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

User management #464

wants to merge 8 commits into from

Conversation

MattIPv4
Copy link
Member

@MattIPv4 MattIPv4 commented Aug 27, 2020

Description

Allow users to be automatically flagged or banned based on the number of invalid PRs that they have. Provide a management ability for Hacktoberfest moderators to be able to manually ban or unban someone.

Test process

TBD.

Requirements to merge

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code in hard-to-understand areas
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@MattIPv4
Copy link
Member Author

MattIPv4 commented Oct 1, 2020

A flaw with the auto-ban, it will re-ban a user if a moderator manually unbans them. Needs to be fixed before this can go to prod.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant