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

[Bug] Synced roles persist upon unverifying #134

Open
EpyonProjects opened this issue Dec 4, 2024 · 4 comments
Open

[Bug] Synced roles persist upon unverifying #134

EpyonProjects opened this issue Dec 4, 2024 · 4 comments
Labels
Severity: Minor Status: Awaiting release Implemented/Fixed, but awaiting release Status: Fixed In Dev Issue has been fixed, but still needs to be released Type: Bug Something isn't working
Milestone

Comments

@EpyonProjects
Copy link

Would it be possible to revert roles gained from the a verified sync upon unverifying? Tough ask, I know.

Given that you can attain in-game MC perms&roles from verifying your account, upon unlinking the account, the in-game player keeps the roles they got from the sync.

If that feature is not possible, how would you go about avoiding this situation?

@hypherionmc hypherionmc added the Type: Bug Something isn't working label Dec 4, 2024
@hypherionmc
Copy link
Owner

Hmmmm shit. That's something that wasn't accounted for, because by the time the "role sync" happens, the account will already be unlinked, so it won't process the roles.

This would be a bug, not a feature request

@EpyonProjects EpyonProjects changed the title [Feature Request] Remove roles upon unverifying [Bug] Synced roles persist upon unverifying Dec 4, 2024
@EpyonProjects
Copy link
Author

Additionally, how are you supposed to trigger the verify command ? I couldn't figure out how to get a code, so I've just been using /staffverify

@hypherionmc
Copy link
Owner

If you log into the server with an unverified account (with access control enabled), you get kicked from the server with the code in the kick message.

We still have to re-add the optional verification, so you can verify your account without needing access control enabled (since access control won't allow you into the server at all without verifying your account)

@EpyonProjects
Copy link
Author

Ah that makes sense. The sync feature was a huge update, so I look forward to what comes next

@hypherionmc hypherionmc added Severity: Minor Status: Fixed In Dev Issue has been fixed, but still needs to be released labels Dec 8, 2024
@hypherionmc hypherionmc added this to the v3.2.3 milestone Dec 10, 2024
@hypherionmc hypherionmc added the Status: Awaiting release Implemented/Fixed, but awaiting release label Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Severity: Minor Status: Awaiting release Implemented/Fixed, but awaiting release Status: Fixed In Dev Issue has been fixed, but still needs to be released Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants