Skip to content

feat(i18n): complete de/en/fr translations and add missing keys - #195

Open
sveneberth wants to merge 1 commit into
viur-framework:developfrom
sveneberth:fix/complete-translations
Open

feat(i18n): complete de/en/fr translations and add missing keys#195
sveneberth wants to merge 1 commit into
viur-framework:developfrom
sveneberth:fix/complete-translations

Conversation

@sveneberth

Copy link
Copy Markdown
Member

Closes #6

Completes the translation dictionary in src/viur/shop/data/translations.py.

What changed

  • French (fr) added for every entry that was missing it.
  • English (en) added for the VAT rate categories (standard, reduced, super_reduced, zero).
  • German (de) added for payment_provider.unzer-paypal.
  • Keys referenced in code but absent from the dict added, so nothing falls back to a raw key/bone name:
    • viur.shop.my_addresses (admin view label in modules/address.py)
    • bone descriptions: AddressSkel.birthdate, ArticleAbstractSkel.shop_view_url, CartItemSkel.frozen_values/is_frozen, CartNodeSkel.frozen_values/is_frozen/total_raw, OrderSkel.email/phone, VatIncludedSkel.value
  • Key order normalized within every entry to _hint, _default_text, then languages alphabetically (de, en, fr), and missing trailing commas added.

Notes

  • French strings were reused from the existing production data where available and sound; the remainder were translated for this change.
  • The SelectBone enum values OrderState and ShippingStatus are intentionally left out: they currently generate generic bare keys (paid, user, …) because no translation_key_prefix is set on those bones. Namespacing them is a separate change and out of scope here.

Fill translation gaps in `data/translations.py`:

- add French (`fr`) for all entries that were missing it
- add English (`en`) for the VAT rate categories
- add German (`de`) for `payment_provider.unzer-paypal`
- add translations for keys referenced in code but absent from the
  dict: the `viur.shop.my_addresses` admin view label and the bone
  descriptions for `birthdate`, `shop_view_url`, `frozen_values`,
  `is_frozen`, `total_raw`, `OrderSkel.email`/`phone` and
  `VatIncludedSkel.value`

Closes viur-framework#6
Copilot AI review requested due to automatic review settings July 24, 2026 23:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sveneberth sveneberth added the Priority: Low This issue can be considered with enough idle time. label Jul 24, 2026
KadirBalku
KadirBalku previously approved these changes Jul 24, 2026
@sveneberth
sveneberth changed the base branch from main to develop July 24, 2026 23:40
@sveneberth
sveneberth dismissed KadirBalku’s stale review July 24, 2026 23:40

The base branch was changed.

@sveneberth
sveneberth enabled auto-merge (squash) July 24, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: translations enhancement New feature or request Priority: Low This issue can be considered with enough idle time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add translations

3 participants