Skip to content

Commit 262dc62

Browse files
committed
don't send email before mentoring is validated
1 parent 49e8e32 commit 262dc62

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

apps/skills/views.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -735,7 +735,6 @@ def contact_mentor(self, request, *args, **kwargs):
735735
raise UserCannotMentorError
736736
serializer = MentorshipContactSerializer(data=request.data)
737737
serializer.is_valid(raise_exception=True)
738-
self.send_email("contact_mentor", skill, **serializer.validated_data)
739738
try:
740739
Mentoring.objects.create(
741740
skill=skill,
@@ -744,6 +743,7 @@ def contact_mentor(self, request, *args, **kwargs):
744743
)
745744
except IntegrityError:
746745
raise DuplicatedMentoringError
746+
self.send_email("contact_mentor", skill, **serializer.validated_data)
747747
return Response(status=status.HTTP_204_NO_CONTENT)
748748

749749
@extend_schema(request=MentorshipContactSerializer, responses={204: None})
@@ -764,7 +764,6 @@ def contact_mentoree(self, request, *args, **kwargs):
764764
raise UserDoesNotNeedMentorError
765765
serializer = MentorshipContactSerializer(data=request.data)
766766
serializer.is_valid(raise_exception=True)
767-
self.send_email("contact_mentoree", skill, **serializer.validated_data)
768767
try:
769768
Mentoring.objects.create(
770769
skill=skill,
@@ -773,4 +772,5 @@ def contact_mentoree(self, request, *args, **kwargs):
773772
)
774773
except IntegrityError:
775774
raise DuplicatedMentoringError
775+
self.send_email("contact_mentoree", skill, **serializer.validated_data)
776776
return Response(status=status.HTTP_204_NO_CONTENT)

0 commit comments

Comments
 (0)