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

Discord Integration! Sync your roles between your NamelessMC site and your Discord Guild. #1764

Merged
merged 23 commits into from
Jul 26, 2020

Conversation

tadhgboyle
Copy link
Member

@tadhgboyle tadhgboyle commented Jul 22, 2020

Will require the Nameless Link bot in your discord guild, which can be easily invited.
Once the bot is in your guild, it will ask the owner to send it a pm which contains the site API URL.
If the url is valid, the bot will listen for role changes and update roles accordingly.
In order for a role change to happen, the affected user needs to have their discord user id set on their nameless site. this can be done by adding it in their profile settings, and then confirming it by PMing the Nameless Link bot their site username.

Still a lil' to do, but making a draft PR so I can add status update comments :p

Major goals:

  • Discord integration option on site (if not enabled, roles will not be synced) [DONE]
  • Discord User ID linking, with extra verification step mentioned above [DONE]
  • Set Discord Role ID for each group [DONE]
  • Website -> Discord role syncing [DONE]
  • Discord -> Website role syncing [DONE]

Minor goals:

  • Error handling when Nameless Link fails to complete a task (verification of Discord ID, etc) [DONE]
  • Give the user their website role in the website's guild when they verify their Discord ID [DONE]

Status: Good to go! Awaiting a final review.

@tadhgboyle
Copy link
Member Author

Update: The Nameless-Link bot is now open source.

core/init.php Outdated Show resolved Hide resolved
core/installation/views/installer.view.php Show resolved Hide resolved
core/installation/views/installer.view.php Show resolved Hide resolved
custom/languages/EnglishUK/admin.php Show resolved Hide resolved
custom/templates/DefaultRevamp/user/settings.tpl Outdated Show resolved Hide resolved
modules/Core/pages/panel/users_edit.php Outdated Show resolved Hide resolved
modules/Core/pages/panel/users_edit.php Show resolved Hide resolved
modules/Core/pages/user/settings.php Outdated Show resolved Hide resolved
modules/Core/pages/validate.php Outdated Show resolved Hide resolved
modules/Core/pages/validate.php Outdated Show resolved Hide resolved
@tadhgboyle tadhgboyle requested a review from samerton July 25, 2020 06:42
@tadhgboyle tadhgboyle marked this pull request as ready for review July 25, 2020 06:45
@tadhgboyle tadhgboyle removed the request for review from samerton July 25, 2020 06:47
@tadhgboyle tadhgboyle merged commit 0560fd9 into v2 Jul 26, 2020
@tadhgboyle tadhgboyle deleted the feature/discord-integration branch July 26, 2020 04:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants