From 447961bd547eed45c11a772b042a8958a78101f9 Mon Sep 17 00:00:00 2001 From: Eric Bailey Date: Tue, 12 Dec 2023 09:37:02 -0600 Subject: [PATCH] Update comment --- src/locale/helpers.ts | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/locale/helpers.ts b/src/locale/helpers.ts index e845233bb4..d80cb5032b 100644 --- a/src/locale/helpers.ts +++ b/src/locale/helpers.ts @@ -95,10 +95,13 @@ export function getTranslatorLink(text: string, lang: string): string { * Contenxt: post-refactor, we populated some user's `appLanguage` setting with * `postLanguage`, which can be a comma-separated list of values. This breaks * `appLanguage` handling in the app, so we introduced this util to parse out a - * valid `appLanguage` from the pre-populated `postLanguage` values. The - * `appLanguage` will be incorrect until the user returns to language settings - * and selects a new option, at which point we'll re-save their choice, which - * should then be a valid option. + * valid `appLanguage` from the pre-populated `postLanguage` values. + * + * The `appLanguage` will continue to be incorrect until the user returns to + * language settings and selects a new option, at which point we'll re-save + * their choice, which should then be a valid option. Since we don't know when + * this will happen, we should leave this here until we feel it's safe to + * remove, or we re-migrate their storage. */ export function sanitizeAppLanguageSetting(appLanguage: string): AppLanguage { const langs = appLanguage.split(',').filter(Boolean)