[WB] Workbook description field is missed

Bug #1428730 reported by Anastasia Kuznetsova on 2015-03-05
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Merlin
Medium
Timur Sufiev

Bug Description

Although workbook definition is optional field, it is presented in YAML workbook definition, but there is no input form to provide it.

Nikolay Makhotkin (nmakhotkin) wrote :

Probably, did you mean 'description'?

Yes, correct

summary: - [Workbook builder] Workbook definition field is missed
+ [Workbook builder] Workbook description field is missed

Happens from time to time on the document reload. Sometimes a description field is missing, sometimes both description and name (or version) are missing. Seems like some bug in Angular.js <-> Barricade.js interaction.

Changed in merlin:
importance: Undecided → Medium
status: New → Triaged
Timur Sufiev (tsufiev-x) on 2015-04-02
Changed in merlin:
status: Triaged → In Progress
assignee: nobody → Timur Sufiev (tsufiev-x)
Timur Sufiev (tsufiev-x) on 2015-04-20
summary: - [Workbook builder] Workbook description field is missed
+ [WB] Workbook description field is missed

Reviewed: https://review.openstack.org/167563
Committed: https://git.openstack.org/cgit/stackforge/merlin/commit/?id=4bc01fe872c3ec27df9b92c4d00b7c9b25190be5
Submitter: Jenkins
Branch: master

commit 4bc01fe872c3ec27df9b92c4d00b7c9b25190be5
Author: Timur Sufiev <email address hidden>
Date: Wed Mar 25 14:09:41 2015 +0300

    Fix the bug with asynchronous template loading

    Due to each field template being loaded asynchronously it was possible
    that some templates hadn't been put to the $templateCache by the time
    they were requested from it for rendering a field directive. This lead
    to some random field shown in the initial document not being rendered
    at all. Fix this problem by using promises, effectively delaying the
    field rendering until the moment the template is finally loaded. Using
    promises also allows to not use $templateCache at all - the
    templateContents are passed as resolve() method argument.

    Also add 'ng-cloak' directive to the toplevel Workbook div to prevent
    raw Angular template flickering during initial load.

    Change-Id: I8a52b9730b52d4dd20400460137576713c081867
    Closes-Bug: #1428730

Changed in merlin:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers