Github user ahgittin commented on the pull request:

    https://github.com/apache/incubator-brooklyn/pull/865#issuecomment-160108239
  
    i've fixed the problem with startup.  now looks much better, and tests work.
    
    however:
    * i thought "Editor" was going to be called "Composer"
    * when using the pop-up wizard there should be a flow by which i can go to 
the composer populated with the YAML for what i've built (this is what the 
`preview` button typically does; maybe we should have the preview enabled even 
on the first step of the wizard if i have something selected)
    * on catalog page, add, if i click "yaml" to add a yaml catalog item, 
nothing happens -- i think it should take me to the blueprint editor, perhaps 
pre-populated with a sample item a la 
http://brooklyn.apache.org/v/latest/ops/catalog/index.html (NB catalog has a 
different syntax than normal deployments; it also accepts the normal deployment 
syntax), or see below
    * do we want a "Help" button on the composer, e.g. with a dropdown that has 
"insert template blueprint", "insert template catalog addition", "go to online 
documentation"
    * on the editor page what is "Delete" supposed to do?  currently seems to 
do nothing.  perhaps remove?  or comment out with TODO if you think it might do 
something in future.  (i'm not sure it does.)
    * on the editor page the "Run" button doesn't work; it constructs a POST 
but it is malformed
    * no feedback is given after clicking "Run"; on success it should take you 
to the "Applications" tab with the app highlighted (you have the app ID in the 
response code so that should be easy), and on failure it should show a message
    * there should be an "Add to Catalog" button on the editor/composer which 
has same feedback after clicking, taking you to catalog page (same as if you 
clicked the old catalog add yaml button)



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to