On 10 February 2014 10:25, Gavin Panella <email address hidden> wrote:
> Couple of minor things:
>
> +def make_json_field(name, label, field_type=None, choices=None, default=None,
> + required=False):
>
> Dosnae matter much, but try using the following form for indentation:
Done. Is that the standard form for MAAS? I was doing it the LP way
but this looks better.
> + """Helper function for build a JSON power type parameters field."""
Actually should have been s/build/building/. Fixed.
On 10 February 2014 10:25, Gavin Panella <email address hidden> wrote: field(name, label, field_type=None, choices=None, default=None,
> Couple of minor things:
>
> +def make_json_
> + required=False):
>
> Dosnae matter much, but try using the following form for indentation:
Done. Is that the standard form for MAAS? I was doing it the LP way
but this looks better.
> + """Helper function for build a JSON power type parameters field."""
Actually should have been s/build/building/. Fixed.