Skip to content

Website: add a French locale - #302

Merged
TanbirRamim merged 3 commits into
TanbirRamim:mainfrom
ggfdrthbf176:feat/fr-locale
Oct 24, 2026
Merged

TanbirRamim merged 3 commits into
TanbirRamim:mainfrom
ggfdrthbf176:feat/fr-locale

Conversation

@ggfdrthbf176

Copy link
Copy Markdown
Contributor

Adds a French (fr-locale) locale to the Open Source Radar website: a full translation of every site message, a language option in the locale selector, and test coverage for the new locale so it stays in sync with English. Also includes the filters.open translation.

Closes #282

  • I didn't edit generated files (issues/by-*, data/, site/data/, the README stats block).
  • For pipeline changes, python3 -m unittest discover scripts, ruff check scripts and ruff format --check scripts pass.

Signed-off-by: ggfdrthbf176 <441563646@qq.com>
Signed-off-by: ggfdrthbf176 <441563646@qq.com>
Signed-off-by: ggfdrthbf176 <441563646@qq.com>
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the pull request, @ggfdrthbf176. Before a maintainer reviews it, one thing needs a look:

Lots of pull requests in a short time [please check]

@ggfdrthbf176 opened 13 pull requests across GitHub in the last 24 hours. The limit set for this repo is 10.

Nothing to change in this PR. A maintainer will take a closer look before reviewing. Fewer, well-tested pull requests usually get merged faster.

Passed: linked issue, issue claim, pull request template, account history, diff scope.

This comment is from pr-doorman, a set of plain rules the maintainers turned on. It doesn't use AI. It updates itself when you edit or push to the pull request.

@ggfdrthbf176

Copy link
Copy Markdown
Contributor Author

Hi @TanbirRamim! I've updated this to the latest main (includes #283 German and the new filters.open sort option) and added the French translation for filters.open. All i18n tests pass locally. This replaces the earlier PR that got auto-closed when I rebuilt the branch. Ready for review, thanks!

@TanbirRamim
TanbirRamim merged commit 3913c48 into TanbirRamim:main Oct 24, 2026
9 checks passed
@TanbirRamim

Copy link
Copy Markdown
Owner

Merged, thanks @ggfdrthbf176, and thanks for the quick rebase! The first test still had a hand-kept locale list, which is why your two PRs kept bumping into each other. #303 makes it loop over every locale, so Bengali only needs its block and the menu option now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Website: add a French locale

2 participants