Page MenuHomePhabricator
Create Task
Maniphest T318327

Decimal separator in Thank you email confirmation fr-FR, es-ES-, it-IT, nl-NL since end of August 2022
Closed, ResolvedPublic
Actions

Description

Hello,

This is following a ticket received by Donor Relations (#1176405), where the donor, Dutch speaking from the Netherlands, complained about the number format, ie. 10.40 rather than 10,40.

This was raised in the past and solved for all relevant countries (see previous phabtask: https://phabricator.wikimedia.org/T293275).

However, checking multiple Civi records over the past two months for donors from France-French, Italy-Italian, Spanish-Spain and Dutch-Netherlands, the number format is switched back to dot/period/full stop instead of comma format since end of August.

Could you please check and update the number format so that the dot is replaced by a comma instead as decimal separator?
For instance: 2.345,90 instead of 2,345.90.

Thanks a lot!

Related Objects

Event Timeline

TomaszGorski created this task.Sep 22 2022, 11:42 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 22 2022, 11:42 AM
XenoRyet moved this task from Triage to Sprint +1 on the Fundraising-Backlog board.Sep 26 2022, 7:52 PM
• TSkaff added a comment.EditedSep 27 2022, 6:41 PM

Per the fr-tech office hours today, adding additional info:

We would like to have, at a minimum, the frFR fix done before France banner launch Oct 4

frFR current display in the TY email:
2.345,90

should be
2,345.90

We can not adjust this ourselves b/c it's pulling from {$amount} (I think :)

Thank you!

XenoRyet triaged this task as High priority.Sep 27 2022, 8:04 PM
XenoRyet moved this task from Sprint +1 to Current Sprint on the Fundraising-Backlog board.
XenoRyet added a project: Fundraising Sprint Recursion Excursion.Sep 27 2022, 8:24 PM
TomaszGorski added a comment.Sep 28 2022, 3:08 AM

Sorry, correction: the current display for frFR in the TY email is 2,345.90 and should be 2.345,90. It should be comma for decimals and dot for thousands instead of what is currently displayed. Thank You!

Eileenmcnaughton subscribed.Sep 28 2022, 3:14 AM

@TomaszGorski - can you keep an eye on whether there is any change from today's outage going forwards. There was some change in the locale handling in the CiviCRM code- although I still need to work through & check how we are interacting with it

TomaszGorski added a comment.Sep 28 2022, 3:19 AM

Will do so, thank you!

TomaszGorski added a comment.Sep 28 2022, 6:11 AM

Hi y'all, I've just seen on Civi two donations made this morning from frFR, and the format is 2,00 €. I suppose all good for France/French! TY Eileen and Thea!

Eileenmcnaughton added a comment.Sep 28 2022, 11:29 PM

Oh yay - thanks for confirming

Damilare assigned this task to Eileenmcnaughton.Sep 29 2022, 3:28 PM
Damilare moved this task from Backlog to Ready for Review on the Fundraising Sprint Recursion Excursion board.
Ejegg moved this task from Ready for Review to Done on the Fundraising Sprint Recursion Excursion board.Oct 4 2022, 4:27 PM
XenoRyet closed this task as Resolved.Oct 11 2022, 7:11 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits