Skip to content

[MSD benchmark] FEATURE: Localization fallbacks (server-side)#9

Open
khaliqgant wants to merge 1 commit into
msd-benchmark-base/discourse/pr-9from
msd-benchmark/discourse/pr-9
Open

[MSD benchmark] FEATURE: Localization fallbacks (server-side)#9
khaliqgant wants to merge 1 commit into
msd-benchmark-base/discourse/pr-9from
msd-benchmark/discourse/pr-9

Conversation

@khaliqgant

Copy link
Copy Markdown

MSD benchmark PR recreated from Martian Code Review Bench.

Source PR: ai-code-review-evaluation#9
Dataset: discourse
Original title: FEATURE: Localization fallbacks (server-side)

Test 9

The FallbackLocaleList object tells I18n::Backend::Fallbacks what order the
languages should be attempted in. Because of the translate_accelerator patch,
the SiteSetting.default_locale is *not* guaranteed to be fully loaded after the
server starts, so a call to ensure_loaded! is added after the locale is set for
the current user.

The declarations of config.i18n.fallbacks = true in the environment files were
actually garbage, because the I18n.default_locale was
SiteSetting.default_locale, so there was nothing to fall back to. *derp*
@khaliqgant khaliqgant changed the base branch from localization-system-pre to msd-benchmark-base/discourse/pr-9 May 15, 2026 14:57
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.

2 participants