On Fri, 2011-09-16 at 11:33 +0200, mfoj...@redhat.com wrote:
> From: Michal Fojtik <mfoj...@redhat.com>
> 
> 
> Signed-off-by: Michal fojtik <mfoj...@redhat.com>

I like doing this a lot, though would like to hear from any API users if
XForms would be useful to them. Or, put differently, I would like to
make sure that what we report there is/can be used by a real-life
client.

Other than that, the patch looks good, with a couple small typos:

> diff --git a/server/views/instances/new.xml.haml 
> b/server/views/instances/new.xml.haml
> new file mode 100644
> index 0000000..498dec2
> --- /dev/null
> +++ b/server/views/instances/new.xml.haml
> @@ -0,0 +1,99 @@
> +%xforms{ :'xmlns:xf' => "http://www.w3.org/2002/xforms";, :'xmlns:xsd' => 
> "http://www.w3.org/2001/XMLSchema";, :'xmlns:xsi' => 
> "http://www.w3.org/2001/XMLSchema-instance"}
> +  %model
> +    %submission{ :id =>"new_instance", :method => "GET",  :action => 
> create_instance_url }

Shouldn't method be POST ?

> +    %instance
> +      %instance

One instance ought to be enough for anybody ;)

David


Reply via email to