Actions directive / delete action hides completion

Bug #1555929 reported by Travis Tripp
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Medium
Tyr Johanson

Bug Description

The angular actions directive and delete service invoke promises, but don't return back a handle back to the promise. This prevents tables using them from just using the standard promise mechanism to know when the action completes. This is especially problematic for making actions truly reusable and blocks the search table from knowing when something completed.

Changed in horizon:
assignee: nobody → Tyr Johanson (tyr-6)
milestone: none → mitaka-rc1
importance: Undecided → Medium
Revision history for this message
Travis Tripp (travis-tripp) wrote :
Changed in horizon:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/290800
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=05b53f7d60c2e9e0c7538348656442a89a03d3fd
Submitter: Jenkins
Branch: master

commit 05b53f7d60c2e9e0c7538348656442a89a03d3fd
Author: Tyr Johanson <email address hidden>
Date: Wed Mar 9 12:22:44 2016 -0700

    Images Actions Return Promises

    The Angular images actions now consistently return a promise. This allows
    a view to fetch an image related action from the registry, call it, and
    have a consistent way to know that it is complete.

    For actions, promises are preferred over events because they require
    less setup and teardown, better handle the asynchronous nature of
    modals, don't require a parent-child scope relationship and finally
    allow for success/fail handler chaining.

    Future patches may add data to the promise so that the view can know
    which items were affected by the action.

    This patch simply returns promises from actions as this is
    a pattern I'd like to establish for Mitaka. Usage of these promises is
    left to a follow-on patch. Also, removing the event mechanism from actions
    is left to a follow-on patch.

    Change-Id: I78d189d6eb026bd98abba6451641010104fc5002
    Partially-Implements: blueprint angularize-images-table
    Closes-Bug: 1555929
    Partial-Bug: 1514627

Changed in horizon:
status: In Progress → Fix Released
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/horizon 9.0.0.0rc1

This issue was fixed in the openstack/horizon 9.0.0.0rc1 release candidate.

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.