Skip to content
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

Fr translation #21

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

alainabbas
Copy link

hello i did some translations in your plugins for French peoples an i set some variables for internationalization of phrases who were in the templates.
hope you will accept them

Regards

<a href="{url}/user/{user.slug}">{user.name}</a> has posted a new reply to "<a href="{url}/topic/{topicSlug}">{title}</a>" in "<a href="{url}/category/{category.slug}">{category.name}</a>"
[[categorynotifications:email-categorie-post,{url},{user.slug},{user.name},{topicSlug},{title},{category.slug},{category.name}]]
Copy link
Member

Choose a reason for hiding this comment

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

7 inclusions is a lot.. could we merge {url}/user/{user.slug}, {url}/topic/{topicSlug}, and {url}/category/{category.slug} into three pairs? Then we'd only have 4 inclusions in the language string.

Copy link
Member

Choose a reason for hiding this comment

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

Three individual language keys, that is.

Copy link
Member

@barisusakli barisusakli Aug 18, 2022

Choose a reason for hiding this comment

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

Do you mean using 3 different language keys instead of one with 7 variables?

Some languages don't follow the same noun/verb structure so sometimes it doesn't work to have 3 keys one after another like [[namespace:key1]] [[namespace:key2]] [[namespace:key3]]

Copy link
Member

Choose a reason for hiding this comment

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

No, I mean the one language key is fine, but the parts making up the three URLs are always going to be the same, so there's no reason to break up the individual variables out.

So instead of [[categorynotifications:email-categorie-post,{url},{user.slug},... you could just do [[categorynotifications:email-categorie-post,{url}/user/{user.slug},...

Copy link
Author

Choose a reason for hiding this comment

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

Ok i m modifying that

@julianlam julianlam requested a review from barisusakli August 18, 2022 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants