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

Give people a ticket-holder role in discord if we can verify their tito ticket #13

Closed
plexus opened this issue Jul 26, 2024 · 3 comments

Comments

@plexus
Copy link
Member

plexus commented Jul 26, 2024

Not 100% decided if we do this through the web app or through the discord bot, the main thing is that we don't want to have to manually check for each person if they have a ticket and give them the right role (which will give access to attendee only channels).

We can get tickets/codes/emails from tito, first step might be to see if there's a ticket on the same email, then we can do it immediately. Otherwise, ask people for their booking code, and then flag that ticket as used.

Careful: orders != tickets, an order can have multiple tickets, and the person doing the order might not be the person who the ticket is assigned to.

@JohnnyJayJay
Copy link
Collaborator

JohnnyJayJay commented Jul 26, 2024

I think it would make sense to put this verification in the web app, this decouples the app from Discord a little. Discord provides a "role connections" API made exactly for the purpose here: you have some Discord-external information that determines whether you get a certain role or not in a server. This is not strictly necessary in our case, we could also simply have the bot add the role as soon as the verification is done. I've already started looking into these options, I'd definitely help here

@JohnnyJayJay
Copy link
Collaborator

The backend work for this is mostly done with #17 and #19. What's still needed is a form somewhere on the frontend that asks users for registration reference + email address. The corresponding endpoint on the backend will then verify that there is a valid ticket (via tito namespace) and then call the role assignment function (in the discord namespace).

@plexus
Copy link
Member Author

plexus commented Aug 17, 2024

I tried this out and it seems to work, I'll make a separate ticket for nicer UI integration.

@plexus plexus closed this as completed Aug 17, 2024
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

No branches or pull requests

2 participants