pycroft icon indicating copy to clipboard operation
pycroft copied to clipboard

Pre Member should have a email

Open agmes4 opened this issue 10 months ago • 1 comments

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

agmes4 avatar Feb 16 '25 15:02 agmes4

I think we should require the email for PreMembers. For this we need to overwrite the email Column for PreMembers

ibot3 avatar Feb 18 '25 14:02 ibot3