-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added email sending feature into direct reservation link create serializer #507
Conversation
543db8d
to
a46629f
Compare
Codecov Report
@@ Coverage Diff @@
## develop #507 +/- ##
===========================================
+ Coverage 75.60% 75.62% +0.01%
===========================================
Files 436 436
Lines 22993 23033 +40
Branches 2306 2309 +3
===========================================
+ Hits 17385 17418 +33
- Misses 5174 5179 +5
- Partials 434 436 +2
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
locale/fi/LC_MESSAGES/django.po
Outdated
@@ -384,6 +384,21 @@ msgctxt "Section type" | |||
msgid "Show if" | |||
msgstr "Näytä, jos" | |||
|
|||
#, fuzzy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove fuzzy
locale/fi/LC_MESSAGES/django.po
Outdated
@@ -3440,6 +3455,18 @@ msgstr "" | |||
"Ilmoitusviestien vastaanottajat. Esimerkki: [email protected],maija@example." | |||
"com" | |||
|
|||
msgctxt "Area search lessor" | |||
msgid "Aluekäyttö- ja valvonta" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Message id should be in English
locale/fi/LC_MESSAGES/django.po
Outdated
@@ -3448,6 +3475,34 @@ msgctxt "Search class" | |||
msgid "Other" | |||
msgstr "Muu kilpailu tai haku" | |||
|
|||
#, fuzzy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove fuzzy
|
||
#, fuzzy | ||
#| msgid "File not available" | ||
msgctxt "Decline reason" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove fuzzy
locale/fi/LC_MESSAGES/django.po
Outdated
msgctxt "Model name" | ||
msgid "Area search sub intended use" | ||
msgstr "Aluehaun alikäyttötarkoitus" | ||
#, fuzzy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove fuzzy
locale/fi/LC_MESSAGES/django.po
Outdated
msgctxt "Model name" | ||
msgid "Area search sub intended uses" | ||
msgstr "Aluehaun alikäyttötarkoitukset" | ||
#, fuzzy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove fuzzy
covering_note = validated_data.pop("covering_note") | ||
send_copy = validated_data.pop("send_copy") | ||
send_email = validated_data.pop("send_mail") | ||
instance = super().create(validated_data) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So this creates DirectReservationLink model without any of information above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes. All this popped information is related to email sending.
instance = super().create(validated_data) | ||
|
||
if send_email: | ||
receivers = [email] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should there be comma?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I remember correctly, black was ranting against it. In tuples comma is needed to make a difference between regular parenthesis
covering_note, | ||
language, | ||
), | ||
None, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be something like "[email protected]"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Default sender is configured in settings.py
a46629f
to
d8b9f94
Compare
d8b9f94
to
85e3d4a
Compare
No description provided.