Skip to content

Addition of missing params#1937

Merged
tdonohue merged 2 commits intoDSpace:mainfrom
pilasou:main
Dec 2, 2022
Merged

Addition of missing params#1937
tdonohue merged 2 commits intoDSpace:mainfrom
pilasou:main

Conversation

@pilasou
Copy link
Copy Markdown
Contributor

@pilasou pilasou commented Oct 28, 2022

Adding missing params and their french translations for : licence approval, SHERPA, external sources and processes.

Adding missing params and their french translations for : licence approval, SHERPA,  external sources and processes.
@tdonohue tdonohue added i18n / l10n Internationalisation and localisation, related to message catalogs 1 APPROVAL pull request only requires a single approval to merge labels Oct 28, 2022
@tdonohue tdonohue added this to the 7.5 milestone Oct 28, 2022
Copy link
Copy Markdown
Contributor

@mhvezina mhvezina left a comment

Choose a reason for hiding this comment

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

Thank you @pilasou for this translation. One comment though: I would use the infinitive and not the imperative tense to continue with the choices made with the existing translation (e.g. "Cliquer" instead of "Cliquez"). What do you think about this?

Comment thread src/assets/i18n/fr.json5 Outdated
Comment thread src/assets/i18n/fr.json5 Outdated
Comment thread src/assets/i18n/fr.json5 Outdated
Comment thread src/assets/i18n/fr.json5 Outdated
Comment thread src/assets/i18n/fr.json5 Outdated
Comment thread src/assets/i18n/fr.json5 Outdated
Comment thread src/assets/i18n/fr.json5 Outdated
New commit to include changes after review of commit 3bf13c0
@tdonohue tdonohue requested review from mhvezina and tdonohue and removed request for mhvezina November 10, 2022 15:58
Copy link
Copy Markdown
Member

@tdonohue tdonohue left a comment

Choose a reason for hiding this comment

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

👍 Thanks @pilasou ! It looks like all feedback from @mhvezina was addressed. I also gave this a quick test just to verify I see the new French translations, and I do. Merging this

@tdonohue tdonohue merged commit 39aee1d into DSpace:main Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge i18n / l10n Internationalisation and localisation, related to message catalogs

Projects

No open projects
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants