mirror of
https://github.com/lingble/chatwoot.git
synced 2025-11-01 19:48:08 +00:00
b989ca6397
# Pull Request Template ## Description This Pull Request will provide a language selector in the Profile Settings for each user, and allows them to change the UI language per agent, defaulting back to the account locale. Fixes # #678 This does PR addresses the Dashboard view but does not change the language of the agents emails ## Type of change Please delete options that are not relevant. - [X ] New feature (non-breaking change which adds functionality) ## How Has This Been Tested? 1. Go to an Agents Profile settings page 2. Select a language from the Language drop down 3. the UI will update to the new i18n locale 4. navigate through the UI to make sure the appropriate language is being used 5. Refresh the page to test that the locale persists 270 - [X] My code follows the style guidelines of this project - [X] I have performed a self-review of my code - [X] I have commented on my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [X] My changes generate no new warnings - [X] I have added tests that prove my fix is effective or that my feature works - [X] New and existing unit tests pass locally with my changes - [X] Any dependent changes have been merged and published in downstream modules Checklist:.724.2708 --------- Co-authored-by: Sojan Jose <sojan@pepalo.com> Co-authored-by: Pranav <pranav@chatwoot.com> Co-authored-by: Sivin Varghese <64252451+iamsivin@users.noreply.github.com> Co-authored-by: iamsivin <iamsivin@gmail.com> Co-authored-by: Muhsin Keloth <muhsinkeramam@gmail.com>
95 lines
2.6 KiB
Ruby
95 lines
2.6 KiB
Ruby
require 'rails_helper'
|
|
|
|
RSpec.describe 'SwitchLocale Concern', type: :controller do
|
|
controller(ApplicationController) do
|
|
include SwitchLocale
|
|
|
|
def index
|
|
switch_locale { render plain: I18n.locale }
|
|
end
|
|
|
|
def account_locale
|
|
switch_locale_using_account_locale { render plain: I18n.locale }
|
|
end
|
|
end
|
|
|
|
let(:account) { create(:account, locale: 'es') }
|
|
let(:portal) { create(:portal, custom_domain: 'custom.example.com', config: { default_locale: 'fr_FR' }) }
|
|
|
|
describe '#switch_locale' do
|
|
context 'when locale is provided in params' do
|
|
it 'sets locale from params' do
|
|
get :index, params: { locale: 'es' }
|
|
expect(response.body).to eq('es')
|
|
end
|
|
|
|
it 'falls back to default locale if invalid' do
|
|
get :index, params: { locale: 'invalid' }
|
|
expect(response.body).to eq('en')
|
|
end
|
|
end
|
|
|
|
context 'when user has a locale set in ui_settings' do
|
|
let(:user) { create(:user, ui_settings: { 'locale' => 'es' }) }
|
|
|
|
before { controller.instance_variable_set(:@user, user) }
|
|
|
|
it 'returns the user locale' do
|
|
expect(controller.send(:locale_from_user)).to eq('es')
|
|
end
|
|
end
|
|
|
|
context 'when user does not have a locale set' do
|
|
let(:user) { create(:user, ui_settings: {}) }
|
|
|
|
before { controller.instance_variable_set(:@user, user) }
|
|
|
|
it 'returns nil' do
|
|
expect(controller.send(:locale_from_user)).to be_nil
|
|
end
|
|
end
|
|
|
|
context 'when request is from custom domain' do
|
|
before { request.host = portal.custom_domain }
|
|
|
|
it 'sets locale from portal' do
|
|
get :index
|
|
expect(response.body).to eq('fr')
|
|
end
|
|
|
|
it 'overrides portal locale with param' do
|
|
get :index, params: { locale: 'es' }
|
|
expect(response.body).to eq('es')
|
|
end
|
|
end
|
|
|
|
context 'when locale is not provided anywhere' do
|
|
it 'sets locale from environment variable' do
|
|
with_modified_env(DEFAULT_LOCALE: 'de_DE') do
|
|
get :index
|
|
expect(response.body).to eq('de')
|
|
end
|
|
end
|
|
|
|
it 'falls back to default locale if env locale invalid' do
|
|
with_modified_env(DEFAULT_LOCALE: 'invalid') do
|
|
get :index
|
|
expect(response.body).to eq('en')
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
describe '#switch_locale_using_account_locale' do
|
|
before do
|
|
routes.draw { get 'account_locale' => 'anonymous#account_locale' }
|
|
end
|
|
|
|
it 'sets locale from account' do
|
|
controller.instance_variable_set(:@current_account, account)
|
|
get :account_locale
|
|
expect(response.body).to eq('es')
|
|
end
|
|
end
|
|
end
|