Skip to content
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

[V3] remove self-bot support #2008

Merged
merged 2 commits into from
Aug 24, 2018
Merged

[V3] remove self-bot support #2008

merged 2 commits into from
Aug 24, 2018

Conversation

mikeshardmind
Copy link
Contributor

Type

  • Bugfix
  • Enhancement
  • New feature

Description of the changes

It's against the terms of service, we tell people not to use them, and we even have special warnings in the code for it. I know a lot of this was prior to discord's stance being firm on this one, but we need to drop the support for this, we shouldn't support API misuse of any kind.

@Redjumpman Redjumpman added the V3 label Aug 12, 2018
@Tobotimus Tobotimus added Type: Removal This intends to remove some functionality. QA: Unassigned and removed QA: Needed labels Aug 13, 2018
@Tobotimus Tobotimus requested a review from Redjumpman August 15, 2018 10:25
@Tobotimus Tobotimus requested review from Tobotimus and removed request for Redjumpman August 23, 2018 14:10
@Tobotimus Tobotimus self-assigned this Aug 23, 2018
@Tobotimus Tobotimus added this to the Beta 20 milestone Aug 23, 2018
Copy link
Member

@Tobotimus Tobotimus left a comment

Choose a reason for hiding this comment

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

Thanks for the patch!

@Tobotimus Tobotimus added QA: Passed Used by few QA members. Has been approved by the assigned QA member(s). and removed QA: Needed labels Aug 24, 2018
@Tobotimus Tobotimus merged commit 6ebfdef into Cog-Creators:V3/develop Aug 24, 2018
@mikeshardmind mikeshardmind deleted the remove-selfbot-support branch December 26, 2019 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QA: Passed Used by few QA members. Has been approved by the assigned QA member(s). Type: Removal This intends to remove some functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants