yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #54766
[Bug 1434926] Re: [Launch Instance Fix] Wizard lifecycle documentation
Fixed elsewhere
** Changed in: horizon
Status: Fix Committed => Invalid
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1434926
Title:
[Launch Instance Fix] Wizard lifecycle documentation
Status in OpenStack Dashboard (Horizon):
Invalid
Bug description:
This is created based on an comment on review patch at:
https://review.openstack.org/#/c/165475/4/horizon/static/angular/wizard/wizard.js
The comment is:
Travis Tripp Mar 20 9:56 PM
Adding the extension hooks is important. I would like to see usage
documentation added on this. We basically need to writeup a Wizard
lifecycle document. An important nuance to this that may trip people
up and should be documented is that it is a broadcast event and the
order of execution between steps is not guaranteed. So critical data
that other steps depend on cannot be reliably handled in this way.
Rather they need to use common model with dynamic data binding (which
is probably the preferred way in most cases so that real time
validation can occur.).
This can be done later as long as it is tracked in etherpad, because
time is very seriously of the essence and I think we'll get a bunch of
changes backed up on this. if you can add some ngdoc sooner than
later, that'd be good.
To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1434926/+subscriptions
References