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

update notification mail in french/english mix #713

Closed
e-dschungel opened this issue Jun 19, 2017 · 1 comment

Comments

@e-dschungel
Copy link

commented Jun 19, 2017

Hi,

I received the new update mail. The header is english, but the text itself is french, see screenshot below.
My guest user language is french [qc] and $conf['browser_language'] is not set in local config so its true by default. My browser language is german.
The language should be the same for both and determined by the browser language or the guest user language.

grafik

@e-dschungel e-dschungel changed the title update notification mail in french update notification mail in french/english mix Jun 19, 2017
@plegall

This comment has been minimized.

Copy link
Member

commented Jun 19, 2017

Hi @e-dschungel yes I saw that too. My "main" text was in English and the email footer was in French. I must check what is wrong.

The language must be the language of the recipient (the admin user).

@plegall plegall self-assigned this Jun 19, 2017
@plegall plegall added this to the 2.9.2 milestone Jun 19, 2017
@plegall plegall closed this in d542de7 Jun 21, 2017
plegall added a commit that referenced this issue Jun 21, 2017
and not only to build the email body message
@plegall plegall added the Type: Bug label Jun 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.