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

🚀 Feature: Limiting Discord login to members of a specific server #338

Closed
axiomen opened this issue Nov 22, 2023 · 6 comments · Fixed by #346
Closed

🚀 Feature: Limiting Discord login to members of a specific server #338

axiomen opened this issue Nov 22, 2023 · 6 comments · Fixed by #346
Labels
feature New feature or request

Comments

@axiomen
Copy link

axiomen commented Nov 22, 2023

🔖 Feature description

Currently, from my understanding, when using Discord login, anyone with a Discord account can login/register and use Pingvin-Share. I'd like to have an option to allow only users from a specific server (guild) to register.

🎤 Pitch

Limiting Discord login only to XYZ server would be a great way to share files with people from the server you want. It would be easier for user management, and the members wouldn't need to have a separate login (people would be more inclined to join and share). Social login for Nextcloud is an example of a plugin that supports it.

@axiomen axiomen added the feature New feature or request label Nov 22, 2023
@zz5840
Copy link
Contributor

zz5840 commented Nov 25, 2023

It's a nice feature. I will implement it soon.

@zz5840
Copy link
Contributor

zz5840 commented Nov 26, 2023

@axiomen Can you test the #346?

@axiomen
Copy link
Author

axiomen commented Nov 28, 2023

@zz5840 Nice, I'll test it today when I have access to my server.

@zz5840
Copy link
Contributor

zz5840 commented Nov 28, 2023

@axiomen Thanks, use stonith404/pingvin-share:development image to test this feature.

@axiomen
Copy link
Author

axiomen commented Nov 28, 2023

@zz5840 Works great! Haven't found any issues.

@zz5840
Copy link
Contributor

zz5840 commented Nov 28, 2023

Giant thanks! We'll release it ASAP.

@axiomen axiomen closed this as completed Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants