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.

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

Throw when attempting to add or remove a member's EveryoneRole #781

Merged
merged 2 commits into from
Aug 17, 2017
Merged

Throw when attempting to add or remove a member's EveryoneRole #781

merged 2 commits into from
Aug 17, 2017

Conversation

foxbot
Copy link
Member

@foxbot foxbot commented Aug 1, 2017

This resolves #780

@foxbot foxbot self-assigned this Aug 1, 2017
@AntiTcb
Copy link
Collaborator

AntiTcb commented Aug 1, 2017

Shouldn't it also be added to other things not valid for everyone? Renaming, deleting, etc.

@Auralytical Auralytical merged commit 506a6c9 into dev Aug 17, 2017
@foxbot foxbot deleted the issues/780 branch August 28, 2017 21:10
FiniteReality pushed a commit to FiniteReality/Discord.Net that referenced this pull request May 5, 2018
@foxbot @FiniteReality
Throw when attempting to add or remove a member's EveryoneRole (disco…
4d44450 
…rd-net#781)

* Throw when attempting to add or remove a member's EveryoneRole

This resolves discord-net#780

* Removed braces
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@foxbot foxbot

Labels
priority: improvement project: rest version: patch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Attempting to add the everyone role to a user causes 404
3 participants
@foxbot @AntiTcb @Auralytical