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

Users are able to request tickets with no maximum after making request #131

Closed
yarunluon opened this issue May 9, 2024 · 4 comments · Fixed by #146
Closed

Users are able to request tickets with no maximum after making request #131

yarunluon opened this issue May 9, 2024 · 4 comments · Fixed by #146
Assignees
Labels
bug-blocking Something isn't working

Comments

@yarunluon
Copy link
Contributor

After making a request and waiting for approval, users can go back into the ticket request and edit the number of tickets requested. There is no maximum bound on the number of possible tickets to request.

image

https://tickets.fnf.events/events/14-fnf-2024-summer-campout/ticket_requests/3020
user: [email protected]
pass: ethical-STRIATED-dormer

@kigster
Copy link
Collaborator

kigster commented May 13, 2024

I believe it's been fixed, but worth verifying.

@beingmattlevy
Copy link
Collaborator

Behavior still there. Can confirm.
I was able to set adults to 10 before it was even approved.

@beingmattlevy beingmattlevy linked a pull request May 13, 2024 that will close this issue
@yarunluon
Copy link
Contributor Author

I'm still experiencing a version of this bug. Use this URL: (ticket_request/3027) to test the behavior. I deleted ticket_request #3020 testing which user roles can delete tickets.

@yarunluon
Copy link
Contributor Author

As of fd9e2ac, users cannot request more than the max allowed. However, it should be noted that all guest names need to be entered for the request to be saved.

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

Successfully merging a pull request may close this issue.

3 participants