On 06/27/2016 04:34 PM, Pavel Vomacka wrote:
> On 06/23/2016 04:25 PM, Petr Vobornik wrote:
>> On 06/20/2016 06:54 PM, Pavel Vomacka wrote:
>>> please review attached patch.
>> 1. I'm not sure whether to include the certificate field in the adder
>> dialog. But if so then it is not good that it accepts different output
>> then a cert widget. The difference is that cert widget is able to work
>> also with "-----BEGIN CERTIFICATE-----" and "-----END CERTIFICATE-----"
> Oh, you are right, I forgot to add custom widget which handles that.
> Fixed now.
>> Otherwise it works good.
>> If we decided not to fix 1. in this patch then ACK.
>> Push should wait on the other cert patches.
> Updated patch attached.
* aaf65e9c56c75d78d1c1f7dcefdb52dd3ddc419a Add certificate widget to ID
override user details page.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code