hunterwilkins2 / bulldog-books

An online bookstore for UGA CSCI 4050. Created with React and Node.
MIT License
2 stars 3 forks source link

Incorrect user type when unsuspending inactive users #67

Closed sking115422 closed 3 years ago

sking115422 commented 3 years ago

I was testing the manage user page today, and i found a bug that if you suspend then unsuspend an inactive user it sets their status to active in stead of back to inactive. This should probably be fixed because this could leave to users with active status with out confirming their email.

sking115422 commented 3 years ago

image

I think we need a conditional statement here to change based on cookie or something

hunterwilkins2 commented 3 years ago

Ummm..... I don't know a good solution to fix this. I don't really want to add another db field to store the previous status of the user. I guess we can change it to make the user's account inactive if they are unsuspended and they will have to reconfirm their email. It's not a great solution but it works. What do you think?