Skip to content

Adding missing french labels#4075

Merged
tdonohue merged 2 commits intoDSpace:mainfrom
pilasou:pilasou-translations-fr
Apr 9, 2025
Merged

Adding missing french labels#4075
tdonohue merged 2 commits intoDSpace:mainfrom
pilasou:pilasou-translations-fr

Conversation

@pilasou
Copy link
Contributor

@pilasou pilasou commented Mar 12, 2025

Addition of french translations for Duplicate detection, ROR and Advanced search features.

Addition of french translations for Duplicate detection, ROR and Advanced search features.
Copy link
Contributor

@nimabehforouz nimabehforouz left a comment

Choose a reason for hiding this comment

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

All French translations reviewed and verified.

Copy link
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.

@pilasou : Could you fix the lint errors in this PR? That way we could merge it. I think I've found the two problems inline below.

"search.filters.operator.equals.text": "Est égal à",

//"search.filters.operator.notequals.text": "Not Equals",
search.filters.operator.notequals.text": "N'est pas égal à",
Copy link
Member

Choose a reason for hiding this comment

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

This line is missing a quote (") at the very beginning

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

"search.filters.operator.authority.text": "Est une autorité",

//"search.filters.operator.notauthority.text": "Not Authority",
"search.filters.operator.notauthority.text": "N'est pas une autorité"
Copy link
Member

Choose a reason for hiding this comment

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

This line is missing a trailing comma (,)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. Thanks @tdonohue for pointing me out the 2 errors.

@tdonohue tdonohue added i18n / l10n Internationalisation and localisation, related to message catalogs 1 APPROVAL pull request only requires a single approval to merge labels Apr 8, 2025
@tdonohue tdonohue moved this to 🙋 Needs Reviewers Assigned in DSpace 9.0 Release Apr 8, 2025
@tdonohue tdonohue added the port to dspace-8_x This PR needs to be ported to `dspace-8_x` branch for next bug-fix release label Apr 8, 2025
@tdonohue tdonohue added this to the 9.0 milestone Apr 9, 2025
Copy link
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 ! This looks good to me as well, and already has approval from @nimabehforouz . So, I'm merging immediately.

@github-project-automation github-project-automation bot moved this from 🙋 Needs Reviewers Assigned to 👍 Reviewer Approved in DSpace 9.0 Release Apr 9, 2025
@tdonohue tdonohue merged commit 57a62b0 into DSpace:main Apr 9, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this from 👍 Reviewer Approved to ✅ Done in DSpace 9.0 Release Apr 9, 2025
@dspace-bot
Copy link
Contributor

Successfully created backport PR for dspace-8_x:

@tdonohue tdonohue removed the port to dspace-8_x This PR needs to be ported to `dspace-8_x` branch for next bug-fix release label May 27, 2025
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.

4 participants