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

[dev.icinga.com #7052] Show translation statistics #953

Closed
icinga-migration opened this issue Sep 2, 2014 · 6 comments
Closed

[dev.icinga.com #7052] Show translation statistics #953

icinga-migration opened this issue Sep 2, 2014 · 6 comments

Comments

@icinga-migration
Copy link

icinga-migration commented Sep 2, 2014

This issue has been migrated from Redmine: https://dev.icinga.com/issues/7052

Created by tgelf on 2014-09-02 08:07:56 +00:00

Assignee: (none)
Status: New
Target Version: Backlog
Last Update: 2016-12-15 09:27:07 +00:00 (in Redmine)


I'd like to see current translation progress on CLI.


@icinga-migration
Copy link
Author

Updated by jmeyer on 2014-09-02 08:22:38 +00:00

Can't this be accomplished with the use of Poedit or something else? At least Poedit displays the current progress in the lower status bar when editing a po-file.

Checking whether there are strings not marked as translatable is a difficult task IMHO because how do we know whether it's really a string that should be translated or not due to it being a name or something similar?

@icinga-migration
Copy link
Author

Updated by tgelf on 2014-09-02 08:38:46 +00:00

Johannes wrote:

Can't this be accomplished with the use of Poedit or something else?

Sure, it's just a feature / nice gimmic. No urgent need for this.

@icinga-migration
Copy link
Author

Updated by elippmann on 2015-03-31 08:37:59 +00:00

  • Target Version set to 2.0.0-rc1

@icinga-migration
Copy link
Author

Updated by jmeyer on 2015-06-11 07:53:25 +00:00

  • Target Version changed from 2.0.0-rc1 to Backlog

@icinga-migration icinga-migration added enhancement New feature or improvement area/localization Affects the localization (i18n / l10n) labels Jan 17, 2017
@icinga-migration icinga-migration added this to the Backlog milestone Jan 17, 2017
nilmerg added a commit that referenced this issue Jan 23, 2017
The separator is not guaranteed to exist.

refs #953
nilmerg added a commit that referenced this issue Jan 23, 2017
@nilmerg
Copy link
Member

nilmerg commented Jan 30, 2017

@theFeu Please fix the following minor issues:

  • Passing an unknown or not enabled module results in an ugly exception. This should rather be a proper error message.
  • Another ugly exception is shown when trying to show statistics for a module without any gettext catalogs. A proper error message seems being more appropriate to me here as well 😉

Once done with it create a pull-request and I'll start with my final review.

theFeu pushed a commit that referenced this issue Feb 7, 2017
theFeu pushed a commit that referenced this issue Feb 7, 2017
@theFeu theFeu assigned nilmerg and unassigned nilmerg Jan 12, 2018
@lippserd lippserd removed this from the Backlog milestone Apr 11, 2018
@nilmerg
Copy link
Member

nilmerg commented Feb 12, 2021

Since https://github.com/Icinga/L10n, this is obsolete I'm afraid.

@nilmerg nilmerg closed this as completed Feb 12, 2021
@nilmerg nilmerg removed area/localization Affects the localization (i18n / l10n) enhancement New feature or improvement labels Feb 12, 2021
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 a pull request may close this issue.

4 participants