Skip to content

🌐(i18n) activate dutch language #742

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 18, 2025
Merged

🌐(i18n) activate dutch language #742

merged 1 commit into from
Mar 18, 2025

Conversation

lunika
Copy link
Member

@lunika lunika commented Mar 17, 2025

Purpose

All the dutch translations are complete on crowdin. We activate it in the django settings and download all translations from crowdin

Proposal

  • 🌐(i18n) activate dutch language

@lunika lunika added the i18n label Mar 17, 2025
@lunika lunika requested review from virgile-dev and AntoLC March 17, 2025 14:11
@lunika lunika self-assigned this Mar 17, 2025
@lunika lunika force-pushed the i18n/activate-dutch branch from 57784f1 to 11c57c1 Compare March 17, 2025 14:11
Copy link
Collaborator

@virgile-dev virgile-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good !

Copy link
Contributor

@berrydenhartog berrydenhartog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lunika lunika force-pushed the i18n/activate-dutch branch from 15f9fee to 7c04b4a Compare March 17, 2025 16:44
Copy link
Collaborator

@AntoLC AntoLC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@lunika lunika force-pushed the i18n/activate-dutch branch from 3963349 to 395920a Compare March 18, 2025 08:12
@lunika lunika enabled auto-merge (rebase) March 18, 2025 08:12
All the dutch translations are complete on crowdin. We activate it in
the django settings and download all translations from crowdin
@lunika lunika force-pushed the i18n/activate-dutch branch from 395920a to 3384bf8 Compare March 18, 2025 08:13
@lunika lunika merged commit 799814e into main Mar 18, 2025
18 of 19 checks passed
@lunika lunika deleted the i18n/activate-dutch branch March 18, 2025 08:27
@AntoLC AntoLC mentioned this pull request Mar 18, 2025
@rvveber
Copy link
Collaborator

rvveber commented Apr 2, 2025

You forgot a migration 🗡️

Everytime a model is modified, even if it's only adding an option, we should create a migration. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants