mistral-dashboard should generate "execute workflow" form dynamically

Bug #1712322 reported by Mateusz Kowalski
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mistral
Fix Released
Undecided
Unassigned

Bug Description

Currently "Execute Workflow" form contains one generic "Input" field which user is expected to fill with correctly structurized dictionary - containing all the input parameters and values required by a workflow definition.

Instead of this Django should dynamically generate list of fields to show in the form so user explicitly sees which parameters are required and which ones can be prefilled with default value.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to mistral-dashboard (master)

Reviewed: https://review.openstack.org/496220
Committed: https://git.openstack.org/cgit/openstack/mistral-dashboard/commit/?id=b42b2c4f25b5e7d95d514323608b481d8c05e232
Submitter: Zuul
Branch: master

commit b42b2c4f25b5e7d95d514323608b481d8c05e232
Author: Mateusz Kowalski <email address hidden>
Date: Tue Aug 22 12:33:51 2017 +0200

    Expand parameter list for workflow execution

    Currently one generic field "Input" is shown when executing a workflow.
    This patch checks list of parameters for a workflow and generates
    a Django form according to the results, also prefilling any default
    value.

    In case default value for a parameter is none, field is not marked
    as required.

    Closes-bug: #1712322
    Change-Id: I702d2b196f87e46bc39da00cad509d2f08f5d24c

Changed in mistral:
status: New → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/mistral-dashboard 6.0.0.0b3

This issue was fixed in the openstack/mistral-dashboard 6.0.0.0b3 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.