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

[Suggestion] Minecraft whitelist for all Discord members #112

Open
canitzp opened this Issue Oct 27, 2017 · 6 comments

Comments

6 participants
@canitzp
Copy link

canitzp commented Oct 27, 2017

The point is that I create a big public modded server, but I don't want to let people rom outside a specific Discord server to join the MC server. I can't whitelist all by hand since there are leaving and joining people permanently, all in all we are 4000 people now at the Discord. It would ba also a good idea, if I can set the ranks a member does need to join.

@andrelec1

This comment has been minimized.

Copy link

andrelec1 commented Oct 27, 2017

Nice idea, but how get miecraft pseudo by Discord_userId ?
There are no official integration of minecraft in discord ... so you don't have this information, and you need to build a liste somewhere with Minecraft pseudo ( or uid ) and discord_userID manualy ... :/

Maybe add some extra code in bot, and people send their minecraft pseudo in commande, and bot make the list ... but he need to crawl the list regulary for removing people loose their grade or leave the discord_guild ...

@Chikachi

This comment has been minimized.

Copy link
Owner

Chikachi commented Nov 19, 2017

Possible way to do this.

Step 1
Player joins server
Step 2
DiscordIntegration checks if Player have Discord account linked
If, continue to step 3
If not, kicks Player with message Type "!link 1234" in #channel on Discord
Step 3
DiscordIntegration checks if Player's Discord account is in guild
If, continue to step 4
If not, kicks Player with message You're not whitelisted
Step 4
DiscordIntegration checks if Player's Discord account have role
If, Player is not kicked
If not, kicks Player with message You're not whitelisted

@thefinaldegree

This comment has been minimized.

Copy link

thefinaldegree commented May 22, 2018

In addition to this, to skip the Step 3 of checking the users Role on Discord, could we have a config option for the bot to assign a rank once the linked?

@Mijago

This comment has been minimized.

Copy link
Collaborator

Mijago commented May 30, 2018

@thefinaldegree IMO, setting the roles should not be forced onto the admin.
The software already has a link feature, so we should use it for this and not use ranks.

As we already check whether or not the user is linked (in Step 2), we do not need step 4.
If the user chooses to link via Discord (by typing !link 1234), he is linked to the bot and we do not need an extra role for it.

Another random idea:
Additionally the bot could check (at connection and while he is connected via an event) whether users are still in the guild or not and delete links from users that are no more on the server. If this works correctly, we do not even need step 3.

@Mijago Mijago added this to To do in Version 3.1.0 via automation Jun 5, 2018

@Mijago Mijago moved this from To do to In Progress in Version 3.1.0 Jun 7, 2018

@Mijago Mijago self-assigned this Jun 7, 2018

@Mijago Mijago moved this from In Progress to To do in Version 3.1.0 Jun 11, 2018

@martmists

This comment has been minimized.

Copy link

martmists commented Jul 2, 2018

My guild runs on a verification-based system, so allowing all linked users with a specific role (configurable) to join would be ideal. In contrast to the idea of the bot assigning a role, it should check for a role we assign ourselves instead.

@thefinaldegree

This comment has been minimized.

Copy link

thefinaldegree commented Nov 19, 2018

@thefinaldegree IMO, setting the roles should not be forced onto the admin.
The software already has a link feature, so we should use it for this and not use ranks.

My apologies, I meant ranks in-game.
Maybe a more generic option would be a "Run this command once linked" on the server?
Then if we're using something like ftbutilities, then one could set the command to be /rank <player> member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment