-
Notifications
You must be signed in to change notification settings - Fork 7
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
Update IT Translaction #35
base: master
Are you sure you want to change the base?
Conversation
@@ -39,5 +39,5 @@ | |||
use robertheim\activitystats\PERMISSIONS; | |||
|
|||
$lang = array_merge($lang, array( | |||
'ACL_'.mb_strtoupper(PERMISSIONS::SEE_STATS) => 'Can see activity stats', |
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.
Please make sure to work on the trunk. You should not use mb_strtoupper and permissions class is lowercase!
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.
Hi,
the good lines should be like that ?
use robertheim\activitystats\permissions;
$lang = array_merge($lang, array(
'ACL_'.utf8_strtoupper(permissions::SEE_STATS) => 'Poò vedere le statistiche di attività',
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.
Yes
Hi @uzz75, could you update this, please ? |
Hi @Galixte Did some update. |
Hi @uzz75, do you want any help to submit a new PR ? |
Hi, Yes. |
yes, you did those changes: uzz75@25d440c. Otherwise, i have submit you a fix, here: uzz75#1 (for this request: #35 (comment)). @RobertHeim, i think it's ok now ? |
@uzz75 accept by merging (merge button in bottom) to fix this mistake. ;)
Fix a mistake
No description provided.