Sounds easy enough to implement.

In T15708#15014, @avivey wrote:

I'd like to see this as an Extension rather then go into the rP, just because we want to have more stuff as extensions; I think this area is abstracted enough so that it can easily be made that way.

That could be a good target as well for this, Is there docs on how extensions should be made?

@valerio.bozzolan & @avivey What is the best practice for setting up the role/server check when new users attempt to auth via the oauth, you can't set limits on the oauth from discords side of view so you'd need to set them up on phorge, I've put down some ideas in the above comment.

In D25513#14722, @aklapper wrote:

Also, apart from the linter results, this patch should not include the changes from D25512

How would i go about implementing it such that for a user to sign in with discord they need to have certain roles.

In D25513#14724, @valerio.bozzolan wrote:

(Small tip: never work in master. Create a git branch for each feature, in sync with origin/master, and with your commits on top of that, so your patch is minimal and arc diff works)