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

500 Server error when 'manual attendance' field is not set #1608

Open
anenadic opened this issue Jan 13, 2020 · 1 comment
Open

500 Server error when 'manual attendance' field is not set #1608

anenadic opened this issue Jan 13, 2020 · 1 comment

Comments

@anenadic
Copy link

@anenadic anenadic commented Jan 13, 2020

Having 'Manual attendance' field unset causes a 500 Server Error. I tried leaving it unset for workshops where the attendance number is not known yet. It defaults to '0' - not sure if this is the desired behaviour (to have attendance at 0). If yes, disregard the issue.

@pbanaszkiewicz

This comment has been minimized.

Copy link
Contributor

@pbanaszkiewicz pbanaszkiewicz commented Jan 20, 2020

Thanks @anenadic for reporting this. I saw bug emails from AMY but didn't know what caused them.

In this case, this is a bug, because the form should always require numeric manual attendance, event if it's 0 by default.

// EDIT: note: suggested solution is to either change blank=True to blank=False in Event.manual_attendance or add required=True for EventForm's field. The former seems better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.