beta-nu-theta-chi / ox-dashboard

A chapter dashabord web app in Django for the Beta Nu Chapter of Theta Chi at Case Western Reserve University
MIT License
0 stars 0 forks source link

Assigning the same meet a brother twice can cause an integrity constraint failure #75

Closed jcassarly closed 3 years ago

jcassarly commented 3 years ago

I was messing around with adding some meet a brothers and added a single brother. When I went to assign meet a brothers, I was able to click the drop down menus to assign the same brother to a candidate twice:

image

Once I submit, I get an integrity error because meet a brothers need to be unique. I think we need some error checking on the meet a brother creation page to yell at the user if they try to create a MAB that already exists and prevent them from doing so.

image

jcassarly commented 3 years ago

Also, Im not sure the meet a brother page has a check to verify the brother is logged in. We should fix that because that has contact info about brothers. I would prefer only brothers have access to that information

helper08 commented 3 years ago

Both of these issues should be fixed now