Closed SaraKalcic closed 3 months ago
@andrezz-b
The requested changes are added. For Yup validation I could not add the error message properly because it would show when you hover over the card rather than when you try to submit without selecting a reason and would disappear when selecting a reason. The validation should still work okay.
@andrezz-b Checkout the changes if you have time, just tagging you in case you are not getting the notifications.
@andrezz-b Checkout the changes if you have time, just tagging you in case you are not getting the notifications.
Yeah, I get a notification on my phone when you tag me, but for other stuff I get an email and sometimes a notification doesn't arrive.
@andrezz-b Checkout the changes if you have time, just tagging you in case you are not getting the notifications.
Yeah, I get a notification on my phone when you tag me, but for other stuff I get an email and sometimes a notification doesn't arrive.
It's okay, I just want to try and see if this pull request can be resolved today. I added the changes you suggested, is everything okay now?
It's fine now, I'll merge it after the presentation.
User reporting should be implemented fully now. So you can just go to a user you matched with and try reporting them, to test it out.