Skip to content

Added support for Phoenix LiveView#20

Open
okuyamacode wants to merge 2 commits intosmeevil:masterfrom
okuyamacode:master
Open

Added support for Phoenix LiveView#20
okuyamacode wants to merge 2 commits intosmeevil:masterfrom
okuyamacode:master

Conversation

@okuyamacode
Copy link
Copy Markdown

To get set_locale working with Phoenix LiveView, the locale needs to be set to the session (as it's running in another thread and Gettext is threadlocal).

Please consider if you like this addition :)

To get set_locale working with Phoenix LiveView, we need to put the locale into the session
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.03%) to 98.333% when pulling 48da86c on raerkeer:master into 68d709f on smeevil:master.

@pierot
Copy link
Copy Markdown
Contributor

pierot commented Jun 15, 2020

@raerkeer
Would you be able to fix the tests so this can be merged? Thanks!

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