Skip to content
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

YJDH-691 | Parametrize youth summer voucher email templates & update for 2024 #2823

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

karisal-anders
Copy link
Collaborator

@karisal-anders karisal-anders commented Feb 10, 2024

Description ✨

feat: parametrize youth summer voucher email templates & update for 2024

also:

  • change employer summer vouchers to use previous year's voucher value
    until 13th of February (i.e. last day before the new year's youth
    summer voucher application period starts)

refs YJDH-691

Issues 🐛

YJDH-691

Testing ⚗️

Finnish youth summer voucher email

Download and rename to fi.eml and you can open this in an email client:

HTML:

  • image

Plaintext:

Swedish youth summer voucher email

Download and rename to sv.eml and you can open this in an email client:

HTML:

  • image

Plaintext:

English youth summer voucher email

Download and rename to en.eml and you can open this in an email client:

HTML:

  • image

Plaintext:

Screenshots 📸

Additional notes 🗒️

@terovirtanen
Copy link
Contributor

API branch is deployed to platta: https://kesaseteli-pr2823.api.dev.hel.ninja/healthz 🚀🚀🚀

@terovirtanen
Copy link
Contributor

EMPLOYER branch is deployed to platta: https://kesaseteli-employer-pr2823.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://kesaseteli-employer-pr2823.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

HANDLER branch is deployed to platta: https://kesaseteli-handler-pr2823.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://kesaseteli-handler-pr2823.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

YOUTH branch is deployed to platta: https://kesaseteli-youth-pr2823.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://kesaseteli-youth-pr2823.dev.hel.ninja 😆🎉🎉🎉

@karisal-anders karisal-anders force-pushed the YJDH-691-update-email-templates branch from 6ab9a27 to e603eed Compare February 12, 2024 11:22
@terovirtanen
Copy link
Contributor

API branch is deployed to platta: https://kesaseteli-pr2823.api.dev.hel.ninja/healthz 🚀🚀🚀

also:
 - change employer summer vouchers to use previous year's voucher value
   until 13th of February (i.e. last day before the new year's youth
   summer voucher application period starts)

refs YJDH-691
@karisal-anders karisal-anders force-pushed the YJDH-691-update-email-templates branch from e603eed to b8dfd3f Compare February 12, 2024 11:43
@karisal-anders karisal-anders marked this pull request as ready for review February 12, 2024 11:44
Copy link

Quality Gate Passed Quality Gate passed for 'yjdh'

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link

Copy link

Copy link

@terovirtanen
Copy link
Contributor

API branch is deployed to platta: https://kesaseteli-pr2823.api.dev.hel.ninja/healthz 🚀🚀🚀

Copy link

@terovirtanen
Copy link
Contributor

EMPLOYER branch is deployed to platta: https://kesaseteli-employer-pr2823.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://kesaseteli-employer-pr2823.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

HANDLER branch is deployed to platta: https://kesaseteli-handler-pr2823.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://kesaseteli-handler-pr2823.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

YOUTH branch is deployed to platta: https://kesaseteli-youth-pr2823.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://kesaseteli-youth-pr2823.dev.hel.ninja 😆🎉🎉🎉

@karisal-anders
Copy link
Collaborator Author

@nikomakela @melniiv @jakezu Added examples of newly generated emails (.eml files) + plaintext versions & screencaptures of the HTML emails for ease-of-review.

@karisal-anders karisal-anders changed the title feat: update for 2024 & parametrize youth summer voucher email templates YJDH-691 | parametrize youth summer voucher email templates & update for 2024 Feb 12, 2024
@karisal-anders karisal-anders changed the title YJDH-691 | parametrize youth summer voucher email templates & update for 2024 YJDH-691 | Parametrize youth summer voucher email templates & update for 2024 Feb 12, 2024
@@ -29,15 +29,12 @@ <h2>{% trans 'Kesäsetelisi, ole hyvä' %}</h2>
<p>
<strong>{{first_name}} {{last_name}}</strong><br/>
<strong>{% trans 'Kesäsetelinumero:' %}</strong> {{summer_voucher_serial_number}}<br/>
<strong>{% trans 'Kesäsetelin summa:' %}</strong> {% trans '350€' %}<br/>
<strong>{% trans 'Kesäsetelin summa:' %}</strong> {{voucher_value_with_euro_sign}}<br/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be much better to give the value as a variable for the translation itself. Then they would be tied together and different locales could represent it differently, for example in different order. Sure this is already in a definition table / list format, so it does not matter that much.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, that's done already in this PR, see definition of voucher_value_with_euro_sign.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean the value comes from a property, and the currency sign (€) is localized to be in front or back of the value per language.

@karisal-anders karisal-anders merged commit 32a0e1c into main Feb 13, 2024
82 checks passed
@karisal-anders karisal-anders deleted the YJDH-691-update-email-templates branch February 13, 2024 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants