Comment 13 for bug 407672

Revision history for this message
Vej (vej) wrote :

@linuxfood I had a look at the code diff and it looks good to me. I especially like your comments and the fact, that you renamed methods to fit better with the new functionality. This is good regarding maintainability in the future.

On thing that I did not checked yet, is the translator support. You introduced a few strings, which are hard to translate (e.g. "A backup automatically starts every %%d %s."). Can you please provide translators with an explanation what %%d and %s are replaced with and if the order of %%d and %s has to be kept or can be changed to fit with style rules of that language.

Best Regards

Vej