Fix form french translation #372

JessyBarrette · 2024-11-28T15:31:56Z

Fix some of the French translation.

Migrate cioos form

github-actions · 2024-11-28T15:35:07Z

Visit the preview URL for this PR (updated for commit dc2d326):

https://cioos-metadata-form-dev-258dc--pr372-370-fix-french-ui-zt9suly7.web.app

_{(expires Wed, 19 Feb 2025 22:12:00 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: c9b6275cb4b6311b719349f5e25e457b5691d09c}

JessyBarrette · 2024-11-28T20:30:23Z

src/components/Pages/PlatformsSaved.jsx

@@ -121,7 +121,7 @@ class Platforms extends FormClassTemplate {
          <Typography variant="h5">
            <I18n>
              <En>Platforms</En>
-              <Fr>Platforms</Fr>
+              <Fr>Plateforme</Fr>


ça semble être au pluriel

JessyBarrette · 2024-11-28T20:35:30Z

src/components/Pages/Submissions.jsx

@@ -203,7 +203,7 @@ class Submissions extends FormClassTemplate {
                    obligatoires, vous pouvez soumettre vos métadonnées pour
                    validation en cliquant sur l’icône « soumettre pour
                    validation ». Vos métadonnées seront publiées lorsqu’elles
-                    auront été validées et approuvées par un professionel du{" "}
+                    auront été validées et approuvées par un professionel de l'{" "}


ce titre est plus large et pas juste l'OGSL et devrait s'appliquer au moins aux régions:
SIOOC du Pacific, SIOOC de l'Atlantique, Amundsen Science ...

Je me demande si on utilise pas: auront été validées et approuvées par un professionel de {" " }

JessyBarrette

Quelques commentaires par rapport aux corrections

src/components/FormComponents/PlatformLeftList.jsx

JessyBarrette · 2024-12-02T13:43:03Z

src/components/NavDrawer.jsx

@@ -44,7 +44,7 @@ import { En, Fr, I18n } from "./I18n";

 import { UserContext } from "../providers/UserProvider";

-const drawerWidth = 240;
+const drawerWidth = 275;


C'est pour la barre de navigation ça?

JessyBarrette · 2024-12-02T13:43:50Z

src/components/NavDrawer.jsx

@@ -175,6 +175,7 @@ export default function MiniDrawer({ children }) {
    signIn: <I18n en="Sign in" fr="Se Connecter" />,
    logout: <I18n en="Logout" fr="Déconnexion" />,
    sharedWithMe: <I18n en="Shared with me" fr="Partagé avec moi" />,
+    envconnection: <I18n en="Connected to development database" fr="Connecté à la base de données  <br /> de développement" />,


Pour suivre la convention des noms de variables en haut il serait bien d'utiliser
envConnection

JessyBarrette · 2024-12-02T13:44:58Z

src/components/NavDrawer.jsx

@@ -466,7 +467,7 @@ export default function MiniDrawer({ children }) {
            )}
          </List>
          <Divider />
-          {usingDevDatabase && <h5>Connected to development database</h5>}
+          {usingDevDatabase && <h5>{translations.envconnection}</h5>}


voir commentaire plus haut

JessyBarrette · 2024-12-02T13:45:54Z

src/regions.js

    },
-    titleFrPossessive: "de l'Observatoire Global du Saint-Laurent (OGSL)",
+    titleFrPossessive: " de l'Observatoire global du Saint-Laurent (OGSL)",


pas sur que l'espace au début est voulu

JessyBarrette · 2024-12-02T17:58:06Z

@kutoso @ABiehlerOGSL et @rodriguezcme!
@bturcotte a fait de nombreux changements dans le formulaire en francais est-ce que vous pouvez réviser le tout et confirmer que ça marche.

Vous pouvez voir les changements sur le site suivante:
https://cioos-metadata-form-dev-258dc--pr372-370-fix-french-ui-f2b3pv0m.web.app/

Migrate CIOOS Metadata

fix/firebase_to_xml dependancies

Fix api for metadata form share

…form into 370-fix-french-ui

JessyBarrette

@bturcotte Quelques commentaires par rapport aux corrections.

Les rôles publisher/editor sont pas mal mélangeant. Je pense que le mieux est de rester transparent et laissé l'utilisateur décider de la signification de chacun des termes.

JessyBarrette · 2025-01-27T18:17:01Z

src/components/FormComponents/InstrumentLeftList.jsx

pas sur que l'espace devrait être là

JessyBarrette · 2025-01-27T18:17:45Z

src/components/FormComponents/LeftList.jsx

@@ -142,7 +142,7 @@ const LeftList = ({
                          title={
                            <I18n
                              en="Duplicate entry"
-                              fr="Duplicate entrée"
+                              fr="Dupliquer"


Double peut-être plus puisque de toutes les autres corrections appliques le mot double

ou plutôt Entrée en double commes les autres corrections

Si je me situe bien dans le formulaire, c'est la fonction pour dupliquer une entrée ? Si oui, je pense que "Dupliquer" serait plus clair que "Entrer en double" qui sonne un peu comme un message d'erreur plutôt que comme une fonctionnalité selon moi.

JessyBarrette · 2025-01-27T18:24:03Z

src/isoCodeLists.js

@@ -84,7 +84,7 @@ export const roleCodes = {
    },
  },
  editor: {
-    title: { en: "Editor", fr: "Éditeur" },
+    title: { en: "Editor", fr: "Réviseur" },


Je suis pas sûr de comprendre pourquoi Editor est pas traduit par Éditeur

je pense que par souscis de clarté je rajouterais le mot anglais à côté

editor: Éditeur (en:editor) publisher: Éditeur (en:publisher)

Justement puisque Editor était traduit par "Éditeur" deux fois alors que c'est deux rôles différents, ont est partie des définitions qui apparaissent en survolant les terme avec :

editor (Party who reviewed or modified the ressource to improve the content) => Réviseur (au lieu de ~~Editeur~~)

publisher (Party who published the ressource) => Éditeur

JessyBarrette · 2025-01-27T18:30:54Z

src/isoCodeLists.js

@@ -143,7 +143,7 @@ export const roleCodes = {
    },
  },
  publisher: {
-    title: { en: "Publisher", fr: "Editeur" },
+    title: { en: "Publisher", fr: "Éditeur" },


voir commentaire plus haut

Merge pull request #371 from cioos-siooc/migrate-cioos-form

a7647ce

Migrate cioos form

JessyBarrette assigned bturcotte and JessyBarrette Nov 28, 2024

bturcotte added 14 commits November 28, 2024 10:35

test UI traduction

74aed7d

changement 28 nov 11h

2e5e339

commit 28 nov 12h05

5517321

commit 28 nov 13h30

f63065f

commit 28 nov 13h45

0e9c311

commit 28 nov 14h

8bef660

Commit 28 nov 14h10

cbf21b1

commit 28 nov 14h25

41b5aa1

commit 28 nov 14h30

c716403

commit 28 nov 14h37

f3b646b

commit 28 nov 14h50

000dc39

commit 28 nov 15h05

c6733d1

commit 28 nov 15h22

1e6c980

15h29

6edb56a

JessyBarrette commented Nov 28, 2024

View reviewed changes

bturcotte added 11 commits November 28, 2024 15:46

commit 15h46

74a1b27

commit 7h38

b651a74

remove bug

4937cfd

commit themes 8h55

31fa24a

commit 9h13

37dec4d

commit 11h15

90478f8

commit 29 nov 13h26

072a165

probleme ds le composant avec des string longues

4c08d56

line break

50d76ef

commit 29 nov 15h

a36aae5

test br

a84c34a

JessyBarrette commented Dec 2, 2024

View reviewed changes

bturcotte added 8 commits December 2, 2024 10:19

commit 2 dec 10h20

4c6ea12

test typography 10h50

0f2aef9

commit 11h

2dbb8ae

commit 11h22

1b62a86

commit 11h30

c4397d0

commit 12h06

a0cf8f8

commit 12h36

bd109e1

commit 12h48

69ded3c

ajout de l'espace qui est nécessaire pour un bel affichage

96b15e3

JessyBarrette removed their assignment Dec 4, 2024

JessyBarrette and others added 7 commits December 6, 2024 09:23

Merge pull request #368 from cioos-siooc/migrate-cioos-form

2aa9271

Migrate CIOOS Metadata

Merge pull request #377 from cioos-siooc/migrate-cioos-form

d2d0f91

fix/firebase_to_xml dependancies

Merge pull request #380 from cioos-siooc/migrate-cioos-form

b7d5dcb

Fix api for metadata form share

commit 20 jan themes list

8194238

commit 20 jan 12h

ba9a25a

Merge branch 'main' of https://github.com/cioos-siooc/metadata-entry-…

3aea4b2

…form into 370-fix-french-ui

commit 20 jan 17h

dc2d326

JessyBarrette commented Jan 27, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix form french translation #372

Fix form french translation #372

JessyBarrette commented Nov 28, 2024

github-actions bot commented Nov 28, 2024 •

edited

Loading

JessyBarrette Nov 28, 2024

JessyBarrette Nov 28, 2024

JessyBarrette left a comment

JessyBarrette Dec 2, 2024

JessyBarrette Dec 2, 2024

JessyBarrette Dec 2, 2024

JessyBarrette Dec 2, 2024

JessyBarrette commented Dec 2, 2024

JessyBarrette left a comment

JessyBarrette Jan 27, 2025

JessyBarrette Jan 27, 2025

JessyBarrette Jan 27, 2025

ABiehlerOGSL Jan 27, 2025

JessyBarrette Jan 27, 2025

JessyBarrette Jan 27, 2025

ABiehlerOGSL Jan 27, 2025

JessyBarrette Jan 27, 2025

Fix form french translation #372

Are you sure you want to change the base?

Fix form french translation #372

Conversation

JessyBarrette commented Nov 28, 2024

github-actions bot commented Nov 28, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

JessyBarrette left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

JessyBarrette commented Dec 2, 2024

JessyBarrette left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Nov 28, 2024 •

edited

Loading