-
Notifications
You must be signed in to change notification settings - Fork 732
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GUACAMOLE-1701: adding French translations #1051
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A minor change below.
You will also need to edit your commit message to correct the spelling error (transalations) and add a period.
extensions/guacamole-auth-restrict/src/main/resources/translations/fr.json
Outdated
Show resolved
Hide resolved
b42674b
to
bbf4237
Compare
Since both commits are related to the same change and their messages are similar, can you squash them into one? |
I believe modifications were done as requested (I am not good with git, sorry...) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last little changes and it will be good!
@@ -10,7 +10,8 @@ | |||
], | |||
|
|||
"translations" : [ | |||
"translations/en.json" | |||
"translations/en.json", | |||
"translations/fr.json", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you remove the comma at the end of this line which break the json?
"TABLE_HEADER_DAY" : "Jour", | ||
"TABLE_HEADER_END_TIME" : "Heure de fin", | ||
"TABLE_HEADER_HOST" : "Hôte", | ||
"TABLE_HEADER_START_TIME" : "Heure de début", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove the comma at the end of this line too?
This PR is created to add French transalations to the new restrict extension.
I've attached it to the main JIRA ticket at the base of this extension, as I don't think this would require a dedicated JIRA issue, but please advise if I 'm wrong.
Best regards
Antoine