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

Multiple concurrent lobbies can be created #89

Open
MasterOfCubesAU opened this issue Jan 28, 2024 · 0 comments · May be fixed by #91
Open

Multiple concurrent lobbies can be created #89

MasterOfCubesAU opened this issue Jan 28, 2024 · 0 comments · May be fixed by #91
Labels
bug Something isn't working

Comments

@MasterOfCubesAU
Copy link
Member

If a user invokes /lobby numerous times sequentially before clicking Create Lobby, once one lobby prompt has been interacted with, the others can be interacted with thus allowing the creation of multiple concurrent lobbies.

This can be fixed by adding a check just before the bot tries to create a lobby in addition to the current check which occurs when a user invokes /lobby

@MasterOfCubesAU MasterOfCubesAU added the bug Something isn't working label Jan 28, 2024
@MasterOfCubesAU MasterOfCubesAU changed the title Duplicate lobbies can be created Multiple concurrent lobbies can be created Jan 28, 2024
@sam1357 sam1357 linked a pull request Jan 28, 2024 that will close this issue
sam1357 added a commit that referenced this issue Aug 19, 2024
Fixes an issue where users are able to create multiple lobbies by loading multiple instances of the starting message.

Closes #89
sam1357 added a commit that referenced this issue Aug 19, 2024
Fixes an issue where users are able to create multiple lobbies by loading multiple instances of the starting message.

Closes #89
sam1357 added a commit that referenced this issue Aug 19, 2024
Fixes an issue where users are able to create multiple lobbies by loading multiple instances of the starting message.

Closes #89
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant