On 05/12/2013 03:10 PM, Ana Krivokapic wrote:
On 05/10/2013 10:37 PM, Endi Sukma Dewata wrote:
On 5/10/2013 9:38 AM, Petr Viktorin wrote:
On 05/10/2013 03:57 PM, Ana Krivokapic wrote:
[...]
Thanks for catching the bugs. Updated patches are attached.


Thanks! It works nicely.
Endi is doing a quick check of the Javascript, if he doesn't find an
issue then ACK.

If this still makes it into 3.2.0, please only push the first patch there.

I tried this in the UI:

   Zone name: test.com
   Authoritative nameserver: ns.sometest.com.

The 'Nameserver IP address' field is still enabled. This is because the name
server is considered in the zone although it's actually not.

The CLI seems to work fine, it didn't ask for IP address.

The UI probably could be fixed using endsWith(ns, '.' + zone). Everything else
looks fine. ACK with the fix.


Fixed, updated patch attached.

ACK for both patches

--
PetrĀ³

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to