Skip to content

Commit

Permalink
fix: add userLocale getter to workaround app config and user locale
Browse files Browse the repository at this point in the history
issue

Signed-off-by: Alexander Trost <[email protected]>
  • Loading branch information
galexrt committed Jan 30, 2025
1 parent 127fb13 commit 8e7068c
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 18 deletions.
12 changes: 6 additions & 6 deletions app/app.vue
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ useSeoMeta({
});

const settings = useSettingsStore();
const { locale: userLocale, isNUIAvailable, design, updateAvailable } = storeToRefs(settings);
const { getUserLocale, isNUIAvailable, design, updateAvailable } = storeToRefs(settings);

if (APP_VERSION !== settings.version) {
logger.info('Resetting app data because new version has been detected', settings.version, APP_VERSION);
Expand All @@ -55,14 +55,14 @@ const onBeforeEnter = async () => {
};

async function setUserLocale(): Promise<void> {
logger.info('Setting user locale to', userLocale.value);
if (userLocale.value !== undefined) {
locale.value = userLocale.value;
await setLocale(userLocale.value);
logger.info('Setting user locale to', getUserLocale.value);
if (getUserLocale.value !== undefined) {
locale.value = getUserLocale.value;
await setLocale(getUserLocale.value);
}
}
setUserLocale();
watch(userLocale, () => setUserLocale());
watch(getUserLocale, () => setUserLocale());

async function clickListener(event: MouseEvent): Promise<void> {
if (!event.target || event.defaultPrevented) {
Expand Down
2 changes: 1 addition & 1 deletion app/components/partials/LanguageSwitcherModal.vue
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@ onMounted(async () => {
languages.value.push({
code: lang.code,
name: lang.name!,
language: lang.language!,
name: lang.name!,
icon: lang.icon ?? 'i-mdi-question',
});
});
Expand Down
21 changes: 11 additions & 10 deletions app/store/settings.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,16 +42,11 @@ export interface SettingsState {
}

export const useSettingsStore = defineStore('settings', {
state: () => {
let defaultLocale = useAppConfig().defaultLocale;
if (defaultLocale === '') {
defaultLocale = 'en';
}

return {
state: () =>
({
version: APP_VERSION,
updateAvailable: false,
locale: defaultLocale,
locale: undefined,

nuiEnabled: false,
nuiResourceName: undefined,
Expand Down Expand Up @@ -85,8 +80,7 @@ export const useSettingsStore = defineStore('settings', {
jobsService: {
cardView: true,
},
} as SettingsState;
},
}) as SettingsState,
persist: {
omit: ['updateAvailable'],
},
Expand All @@ -106,6 +100,13 @@ export const useSettingsStore = defineStore('settings', {
isNUIAvailable(state): boolean {
return state.nuiEnabled ?? false;
},
getUserLocale(state): Locale {
return state.locale !== undefined
? state.locale
: useAppConfig().defaultLocale !== ''
? (useAppConfig().defaultLocale as Locale)
: 'en';
},
},
});

Expand Down
1 change: 0 additions & 1 deletion config.example.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,6 @@ discord:
game:
# First grade "number/id", some servers start at 0, but I have seen some start at 1
startJobGrade: 0
defaultLocale: "en"
sync:
enabled: false
apiTokens: []
Expand Down

0 comments on commit 8e7068c

Please sign in to comment.