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

Set the scope to also be able to retrieve the user's email #122

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lra
Copy link
Member

@lra lra commented Sep 29, 2024

Without this, we can't contact anyone for notifications or moderation events.

The problem with that is that in order to update the scope for a user, the user has to disconnect pouet from https://id.scene.org/profile/ and I don't see a way for Pouët to update or invalidate the existing SceneID pouet login to update the scope.

Do you?

Without this, we can't contact anyone for notifications or moderation events.
@stijnstijn
Copy link

I don't really want Pouët notifications via e-mail. Generally speaking and specifically here, I would prefer not sharing my e-mail if not necessary, and I think it's neat how I can use Pouët while sharing a very minimal set of data (not even including my e-mail!). Could this not be an opt-in feature? Especially because currently nothing on Pouët would actually use this data, so requiring people to share it feels like putting the cart before the horse.

@lra
Copy link
Member Author

lra commented Jan 11, 2025

Pouet needs the emails of its users, or there is no way to auth those users as it has no access to the scene ID database.
So if the link to sceneID is ever broken, we have a useless database of users.

It's not to spam users with notifications, it's to

  1. authenticate users to avoid impersonations (right now it's not possible and I have to trust people)
  2. let the team be able to contact them privately if there is a need to

The previous maintainer had access to both, it's not the case anymore and it's a problem.

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.

2 participants