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 possibility to change locale from login/register pages #3216
Conversation
nicosomb
commented
Jun 12, 2017
•
edited by j0k3r
edited by j0k3r
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Deprecations? | no |
Tests pass? | no |
Documentation | no |
Translation | no |
Fixed tickets | #1695 |
License | MIT |
- test ?
*/ | ||
public function setLocaleAction(Request $request, $language = null) | ||
{ | ||
if ($language != null) { |
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.
!==
5afe60e
to
e4dcc1b
Compare
e4dcc1b
to
9d5f120
Compare
I rebased this PR. @j0k3r Do you think that we need test for this one? I can't test the language displayed on the login form because in our testsuite, we don't have language. |
Languages are hard coded in the HTML because we want the word in german, english and french (and not "Allemand, Anglais, Français" for example). |
9d5f120
to
17f120e
Compare
- Do not override locale if user has choosen a locale from the login screen. - Add some tests about locale url
17f120e
to
4d4147b
Compare
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.
LGTM