bito-code-review[bot] commented on code in PR #41752:
URL: https://github.com/apache/superset/pull/41752#discussion_r3521426064
##########
superset/translations/fr/LC_MESSAGES/messages.po:
##########
@@ -12393,11 +12331,8 @@ msgstr "Sélectionner un tableau de bord"
msgid "Select a database"
msgstr "Sélectionner la base de données"
-#, fuzzy
msgid "Select a database table and create dataset"
-msgstr ""
-"Sélectionner une tableau de base de données et créer un ensemble de "
-"données"
+msgstr "Sélectionner une tableau de base de données et créer un jeu de données"
Review Comment:
<!-- Bito Reply -->
The suggestion to correct the grammatical gender mismatch is appropriate and
improves the accuracy of the French translation. Applying this change ensures
the UI label correctly uses the feminine noun 'table' with the matching article.
**superset/translations/fr/LC_MESSAGES/messages.po**
```
msgid "Select a database table and create dataset"
-msgstr "Sélectionner une tableau de base de données et créer un ensemble de
données"
+msgstr "Sélectionner une table de base de données et créer un jeu de
données"
```
##########
superset/translations/fr/LC_MESSAGES/messages.po:
##########
@@ -15485,15 +15416,13 @@ msgstr "Une erreur s'est produite lors de la
récupération des schémas"
msgid "There was an error creating the user. Please, try again."
msgstr "Une erreur s'est produite lors de la récupération des schémas"
-#, fuzzy
msgid "There was an error duplicating the role. Please, try again."
-msgstr "Il y a eu un problème de duplication de l'ensemble de données."
+msgstr "Il y a eu un problème de duplication du jeu de données."
Review Comment:
<!-- Bito Reply -->
The updated translation correctly addresses the issue by using 'jeu de
données' instead of 'rôle' in the context of duplicating a dataset, which
aligns with the intended meaning. The change is appropriate and resolves the
translation error identified in the review.
**superset/translations/fr/LC_MESSAGES/messages.po**
```
msgid "There was an error duplicating the role. Please, try again."
msgstr "Il y a eu un problème de duplication du jeu de données."
```
##########
superset/translations/fr/LC_MESSAGES/messages.po:
##########
@@ -15645,39 +15573,30 @@ msgstr ""
msgid "There was an issue deleting: %s"
msgstr "Il y a eu un problème lors de la suppression de : %s"
-#, fuzzy, python-format
+#, python-format
msgid "There was an issue duplicating the %s."
-msgstr "Il y a eu un problème de duplication de l'ensemble de données."
+msgstr "Il y a eu un problème de duplication avec %s."
-#, fuzzy, python-format
+#, python-format
msgid "There was an issue duplicating the selected %s: %s"
-msgstr "Il y a eu un problème lors de la suppression des %s sélectionné(e)s
:%s"
+msgstr "Il y a eu un problème de duplication avec les %s sélectionné(e)s : %s"
-#, fuzzy, python-format
+#, python-format
msgid "There was an issue exporting the %s"
-msgstr "Il y a eu un problème de duplication de l'ensemble de données."
+msgstr "Il y a eu un problème durant l'export de %s"
-#, fuzzy, python-format
+#, python-format
msgid "There was an issue exporting the selected %s"
-msgstr ""
-"Il y a eu un problème lors de la suppression des modèles sélectionnées : "
-"%s"
+msgstr "Il y a eu un problème durant l'export des éléments sélectionné(e)s :
%s"
Review Comment:
<!-- Bito Reply -->
The decision to retain the inclusive form is appropriate. Since the source
text uses a consistent pattern and the inclusive form is a deliberate choice
for gender neutrality in this context, maintaining the existing style is a
reasonable approach.
##########
superset/translations/fr/LC_MESSAGES/messages.po:
##########
@@ -15571,11 +15500,10 @@ msgstr ""
"Désolé, une erreur s'est produite lors de la récupération des "
"informations de cet ensemble de données"
-#, fuzzy
msgid "There was an error while fetching users"
msgstr ""
"Désolé, une erreur s'est produite lors de la récupération des "
-"informations de cet ensemble de données"
+"informations de ce jeu de données"
Review Comment:
<!-- Bito Reply -->
The user has addressed the reviewer's suggestion regarding the translation
of 'There was an error while fetching users'. The translation has been updated
to 'récupération des utilisateurs' as requested.
**superset/translations/fr/LC_MESSAGES/messages.po**
```
msgid "There was an error while fetching users"
msgstr ""
-"Désolé, une erreur s'est produite lors de la récupération des "
-"informations de cet ensemble de données"
+"Désolé, une erreur s'est produite lors de la récupération des "
+"informations de ce jeu de données"
```
##########
superset/translations/fr/LC_MESSAGES/messages.po:
##########
@@ -18270,13 +18184,12 @@ msgstr ""
"L'écrasement pourrait vous faire perdre une partie de votre travail. "
"Êtes-vous sûr de vouloir les écraser?"
-#, fuzzy
msgid ""
"You are importing one or more themes that already exist. Overwriting "
"might cause you to lose some of your work. Are you sure you want to "
"overwrite?"
msgstr ""
-"Vous importez un ou plusieurs ensembles de données qui existent déjà. "
+"Vous importez un ou plusieurs jeux de données qui existent déjà. "
Review Comment:
<!-- Bito Reply -->
The suggestion provided by the reviewer was accurate and necessary to
correct the translation. The original message was incorrectly using the
terminology for datasets instead of themes, which would have caused confusion
for users importing themes. Applying this correction ensures the translation is
contextually accurate.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
