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

LeaderOS email verification support #136

Open
05678 opened this issue Apr 26, 2024 · 11 comments
Open

LeaderOS email verification support #136

05678 opened this issue Apr 26, 2024 · 11 comments
Labels
enhancement New feature or request Priority: Medium issue/suggestion

Comments

@05678
Copy link

05678 commented Apr 26, 2024

What feature do you want to see added?

When the LeaderOS uses the email verification, you can't access the user settings and other things in the web before you verify your email. But If I connect the authmerereloaded and the leaderos, like they are saving in same database (LeaderOS has authme support no worries) when the user registers on the web, the leaderos adds the user to the database, and the user can login with their password. I need a system that checks the isVerified (0 - not verified, 1 - verified) value in the database at the user's data, and if the user is not verified, than kicks it from the server.

Are there any alternatives?

I edit the plugin myself

Anything else?

I really hope you can do that, because this is a good thing, you can call it LeaderOS email verification support.

Thank you for your answer.

@05678 05678 added the enhancement New feature or request label Apr 26, 2024
@05678
Copy link
Author

05678 commented Apr 26, 2024

And if you want to do this, I can give you a LeaderOS api.

@HaHaWTH
Copy link
Owner

HaHaWTH commented Apr 26, 2024

A value that determine whether the player is verified his email?

@05678
Copy link
Author

05678 commented Apr 27, 2024

Yos

@05678
Copy link
Author

05678 commented Apr 27, 2024

So you just need to check it at the player login event

@05678
Copy link
Author

05678 commented Apr 27, 2024

And cancel the event if the value is false

@HaHaWTH
Copy link
Owner

HaHaWTH commented Apr 27, 2024

I may create a new branch for this

@05678
Copy link
Author

05678 commented Apr 27, 2024

Thank you so much!

@05678
Copy link
Author

05678 commented Apr 29, 2024

Sorry, but are you going to do that, or just create a branch?

@HaHaWTH
Copy link
Owner

HaHaWTH commented Apr 29, 2024

Working on it, when stable will push this. Maybe a bit slow because I need more tests and keep compatibility:(

@HaHaWTH
Copy link
Owner

HaHaWTH commented Apr 30, 2024

We need to sync with upstream changes currently, they may drop the support of old versions

@05678
Copy link
Author

05678 commented Apr 30, 2024

Thank you, have a nice day

@HaHaWTH HaHaWTH added the Priority: Medium issue/suggestion label May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Priority: Medium issue/suggestion
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

2 participants