HUE-1267 [oozie] Import action from josub or a workflow

Review Request #3115 — Created June 5, 2013 and submitted

abec
old-hue-rw
HUE-1267
hue
enricoberti, kevin.wang, romain
commit 8de00b49b3c6844d0cca70815de82708efe21dab
Author: Abraham Elmahrek <abraham@elmahrek.com>
Date:   Tue Jun 4 13:59:50 2013 -0700

    HUE-1267 [oozie] Import action from josub or a workflow
    
    - Created new control that manages import actions.
    - Can import from jobsub or oozie
    - Jobsub import is just like oozie import, except oozie requires action selection

:100644 100644 93108f7... bf55bba... M	apps/oozie/src/oozie/models.py
:100644 100644 86161d0... 65e4d1a... M	apps/oozie/src/oozie/templates/editor/action_utils.mako
:100644 100644 30a6051... b97cb4d... M	apps/oozie/src/oozie/templates/editor/edit_workflow.mako
:100644 100644 095b386... 76e216d... M	apps/oozie/src/oozie/urls.py
:100644 100644 68e0935... 1d7a755... M	apps/oozie/src/oozie/views/api.py
:100644 100644 3f70edb... e286354... M	apps/oozie/static/css/workflow.css
:000000 100644 0000000... 127cdbc... A	apps/oozie/static/js/workflow.import-node.js
:100644 100644 846de11... f350c33... M	apps/oozie/static/js/workflow.js
:100644 100644 8ced060... 68d0dfd... M	apps/oozie/static/js/workflow.registry.js
:100755 100755 8116a4d... 50499d5... M	desktop/core/static/js/Source/jHue/jquery.rowselector.js
Manually tested.
Ran oozie tests.
  • 0
  • 0
  • 3
  • 0
  • 3
Description From Last Updated
romain
  1. A bit of scary stuff but not that bad at all!!
  2. apps/oozie/src/oozie/models.py (Diff revision 1)
     
     
    Maybe request.user --> user instead to decouple a bit
  3. Maybe remove ' and add it to the beginning of the flow' to make it cleaner here
  4. prepended to your workflow -->  added to the beginning of the flow
    
    Probably easier to understand for the users
  5. double space after 'id'
  6. 
      
abec
abec
Review request changed

Status: Closed (submitted)

Loading...