Hi Brian,

> I noticed that the 'add account' dialog was changed substantially, and 
> converted to a custom UI widget. Additionally, the password
> confirm field validation is broken, so I am unable to add a new account

The change was made in order to allow the wizard to change to include
a list of users grabbed from LDAP when LDAP is enabled. This was done
in order to aid ease of user provisioning.

The change was discussed some time ago on the mailing list:
Switch dialog actions based on result from API call. -
http://markmail.org/thread/ebtm2iqzvz3sm3ne
UI Query: Create Form - http://markmail.org/thread/kcngmhlgxeukvscb

Based on the replies from Jessica and Pranav the only way to go about
implementing this was via the custom widget.

> This doesn't seem to really need something custom from the looks of it

Is there a way we can use createForm and still implement the custom
dialog ( http://ianduffy.ie/cloudstack/screen1.png ) when Ldap is
enabled?

Very sorry about this! Wasn't aware such issue was present.

Thanks,
Ian

On 10 October 2013 21:03, Jessica Wang <jessica.w...@citrix.com> wrote:
> +1
>
> -----Original Message-----
> From: Brian Federle
> Sent: Thursday, October 10, 2013 12:55 PM
> To: dev@cloudstack.apache.org
> Cc: Animesh Chaturvedi; Jessica Wang; Sonny Chhen
> Subject: RE: UI accounts wizard broken
>
> Also, I am referring to this commit. I do not remember any active UI 
> comitters including myself receiving a review request for this...
>
> commit eaa41433715ba1a1137ad9f346ea8569e31e4560
> Author: Ian Duffy <i...@ianduffy.ie>
> Date:   Thu Jul 25 10:24:13 2013 +0100
>
>     Merge LDAPPlugin
>
>     Signed-off-by: Abhinandan Prateek <aprat...@apache.org>
>
> -----Original Message-----
> From: Brian Federle [mailto:brian.fede...@citrix.com]
> Sent: Thursday, October 10, 2013 12:46 PM
> To: dev@cloudstack.apache.org
> Cc: Animesh Chaturvedi; Jessica Wang; Sonny Chhen
> Subject: UI accounts wizard broken
>
> After reviewing the UI on the latest master, I noticed that the 'add account' 
> dialog was changed substantially, and converted to a custom UI widget. 
> Additionally, the password confirm field validation is broken, so I am unable 
> to add a new account.
>
> Can I know the reason this was converted to a custom widget, instead of using 
> the standard createForm? This doesn't seem to really need something custom 
> from the looks of it. I am mainly concerned since it is causing validation 
> bugs as well as some display glitches, which could be avoided by using the 
> standard widget toolkit.
>
> Thanks,
> Brian

Reply via email to