pycroft
pycroft copied to clipboard
Pre Member should have a email
Bug
when trying to add a pre member for testing purposes I encountered a bug in which I just got a 500 when request the membership request. This happend cause I did not set a mail address. Its in schema defined that a pre member does not need a email. I think this is counter intuitive.
fix
I see two options in order for it to make more sense
- remove the is nullable
- remove the check in the request
/user/json/member-requests
I think we should require the email for PreMembers. For this we need to overwrite the email Column for PreMembers