Skip to content

Fix: Corriger les clés de traduction brutes - #114

Merged
bahmez merged 1 commit into
devfrom
bugfix/fix-translation
Feb 1, 2026
Merged

bahmez merged 1 commit into
devfrom
bugfix/fix-translation

Conversation

@bahmez

@bahmez bahmez commented Jan 31, 2026

Copy link
Copy Markdown
Owner

Résumé

Résolution de deux problèmes d'affichage de clés de traduction brutes :

  • Les toasts d'erreur affichaient members.impossible et members.cannot_remove_last_admin
  • La description de la "Zone de danger" affichait settings.archive_description

Modifications

  • apps/web/public/locales/fr/common.json
  • apps/web/public/locales/en/common.json

Problème

Les fichiers contenaient des blocs members et settings dupliqués au début et à la fin. Les composants utilisaient les clés du bloc final, qui n'étaient pas correctement résolues.

Solution

  • Suppression des doublons
  • Fusion de tous les blocs en un seul par section
  • Amélioration des messages d'erreur : ajout de "C'est une limitation volontaire."

Test

  1. Aller dans Membres avec un seul admin
  2. Essayer de retirer l'admin → le toast affiche un message lisible
  3. Aller dans Paramètres → la description de la zone de danger s'affiche correctement
  4. Vérifier les deux langues (FR/EN)

…sections

- Restored and enhanced translations for members and settings sections in both English and French.
- Added clarifications regarding limitations on removing the last admin and member.
- Ensured consistency in workspace selection prompts across locales.
@bahmez
bahmez requested a review from schekina2 January 31, 2026 22:47
@bahmez bahmez self-assigned this Jan 31, 2026
@schekina2

Copy link
Copy Markdown
Collaborator

Correction OK : suppression des doublons de traductions, clés correctement résolues.
Messages d’erreur lisibles, affichage conforme en FR/EN.
Tests fonctionnels validés.

@bahmez
bahmez merged commit 77b0f2c into dev Feb 1, 2026
1 check passed
@bahmez
bahmez deleted the bugfix/fix-translation branch February 1, 2026 22:51
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