-
Notifications
You must be signed in to change notification settings - Fork 16
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
base: master
Are you sure you want to change the base?
Fr translation #21
Conversation
<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}]] |
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.
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.
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.
Three individual language keys, that is.
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.
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]]
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.
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},...
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.
Ok i m modifying that
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