Hello, Thank you for your patch as well.
>- doc=_('Reason for revoking the certificate (0-10)'), >+ doc=_('Reason for revoking the certificate (0-10). See RFC 5280 (paragraph 5.3.1) for reason details'), Rather than just specifying the RFC with the paragraph to go look up, can you either add the revocation options or say something like: + doc=_('Reason for revoking the certificate (0-10). See \'ipa help cert\' for revocation reason details.'), IMO, it is a little annoying to go look up revocation reasons when those reasons can either be added to the help output or exist already in `ipa help cert`. Thanks, Gabe On Tue, May 3, 2016 at 8:13 AM, Martin Basti <mba...@redhat.com> wrote: > > > On 03.05.2016 16:01, Patrice Duc-Jacquet wrote: > > Hi everyone > this is my first patch. So I may have done thhings nor in a proper way. > Please let me know if something is wrong in the proceess I followed. With > regards > > Pat > > > Hello, > > thank you for your patch. Please remove changes in .po and .pot files from > the patch, these files are generated automatically from zanata. > > thank you > > Martin > > -- > Manage your subscription for the Freeipa-devel mailing list: > https://www.redhat.com/mailman/listinfo/freeipa-devel > Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code >
-- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code