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

Make some menus translatable #12885

merged 2 commits into from Jun 10, 2018


Copy link

@c960657 c960657 commented May 10, 2018

Fix some menu items that are not translatable.

@sgiehl sgiehl added Needs Review For pull requests that need a code review. c: i18n For issues around internationalisation and localisation. labels May 10, 2018
Copy link

tsteur commented May 10, 2018

fyi in theory changing Piwik Mobile App => Mobile_MatomoMobile' is somewhat breaking API so we should at least mention this in the developer changelog.

Why is this breaking API? Developers can hide menus using the menuItem, submenuItem name (translation key). If previously something was removed before based on that specific name, it will no longer work afterwards.

I don't think for those menu items it will be a big deal, but we might need to mention it?

Copy link
Contributor Author

c960657 commented May 11, 2018

Ah, I didn't know that. I have added a note to the changelog.

@sgiehl sgiehl added this to the 3.6.0 milestone May 15, 2018
@c960657 c960657 force-pushed the translate-menu branch 3 times, most recently from 79d1f78 to cb1347d Compare May 22, 2018 17:10
@diosmosis diosmosis merged commit 6666479 into matomo-org:3.x-dev Jun 10, 2018
@c960657 c960657 deleted the translate-menu branch June 11, 2018 06:29
InfinityVoid pushed a commit to InfinityVoid/matomo that referenced this pull request Oct 11, 2018
* Make some menus translatable

* Update changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
c: i18n For issues around internationalisation and localisation. Needs Review For pull requests that need a code review.
None yet

Successfully merging this pull request may close these issues.

None yet

4 participants