summaryrefslogtreecommitdiffstats
path: root/cvp/3rd_party/static
diff options
context:
space:
mode:
authorgrakiss <grakiss.wanglei@huawei.com>2017-11-01 08:00:40 +0000
committergrakiss <grakiss.wanglei@huawei.com>2017-11-01 08:55:02 +0000
commitb3501de5a645f69c705e64345965aab9cd271355 (patch)
tree537bd2218f51f4f9ff77bf8af79cf80e833dbcb3 /cvp/3rd_party/static
parent55bb29b1d7b977b387a74f754b4baea94274141c (diff)
Remove approve / no approve options for reviewers
Distinguish results shared or submitted from a tester JIRA: DOVETAIL-544 JIRA: DOVETAIL-545 [cvp-web]There is consensus within C&C committee to move towards a simpler means of voting by reviewers. We will use email voting (+1) on the [cvp@opnfv.org|mailto:cvp@opnfv.org] email alias. For now, can we disable the approve / not approve options in the drop-down in the 'Operation' column of the results? We can revisit this feature in the future after some experience with the manual method. [cvp-web]When a user shares or submits results, the recipient of those results (share target or reviewer) do not know where the results originated from. To remedy this issue, we will add a column labeled 'Owner' to the 'My Results' table that shows the email address or the sending party? Change-Id: I586b01ad113d898fe3d5824025df9877118fa622 Signed-off-by: grakiss <grakiss.wanglei@huawei.com>
Diffstat (limited to 'cvp/3rd_party/static')
-rw-r--r--cvp/3rd_party/static/testapi-ui/components/results/results.html6
1 files changed, 6 insertions, 0 deletions
diff --git a/cvp/3rd_party/static/testapi-ui/components/results/results.html b/cvp/3rd_party/static/testapi-ui/components/results/results.html
index cd38ad75..9127e170 100644
--- a/cvp/3rd_party/static/testapi-ui/components/results/results.html
+++ b/cvp/3rd_party/static/testapi-ui/components/results/results.html
@@ -21,6 +21,7 @@
<tr>
<th>Upload Date</th>
<th>Test ID</th>
+ <th>Owner</th>
<th>File Name</th>
<th>Label</th>
<th>Status</th>
@@ -35,6 +36,8 @@
<tr ng-repeat="(index, result) in ctrl.data.tests">
<td>{{ result.upload_date | limitTo:19}}</td>
<td><a ng-click="ctrl.gotoResultDetail(result.id, result._id)">{{ result.id }}</a></td>
+ <td><a uib-tooltip="{{ result.id }}" tooltip-placement="top" tooltip-append-to-body="true" ng-click="ctrl.gotoResultDetail(result.id, result._id)">{{ result.id | limitTo:8 }}</a></td>
+ <td>{{ result.owner }}</td>
<td>{{ result.filename || "None"}}</td>
<td><div class="popover-wrapper"><a editable-theme="bs3" onbeforesave="ctrl.toggleCheck(result, 'label', $data)" editable-text="result.label"> {{ result.label || "None" }}</a></div></td>
<td>{{ result.status }}</td>
@@ -48,8 +51,10 @@
<ul class="dropdown-menu" uib-dropdown-menu role="menu" aria-labelledby="single-button">
<li role="menuitem" ng-if="auth.currentUser.role == 'user'" class="menu-item menu-item-type-post_type menu-item-object-page"><a ng-class="{'hide': result.status != 'review'}" ng-click="ctrl.toPrivate(result, 'private')">withdraw submit</a></li>
<li role="menuitem" ng-if="auth.currentUser.role == 'user'" class="menu-item menu-item-type-post_type menu-item-object-page"><a ng-class="{'hide': result.status != 'private'}" ng-click="ctrl.toReview(result, 'review')">submit to review</a></li>
+ <!--
<li role="menuitem" ng-if="auth.currentUser.role.indexOf('reviewer') != -1" class="menu-item menu-item-type-post_type menu-item-object-page"><a ng-class="{'hide': result.status == 'approved'}" ng-click="ctrl.toggleCheck(result, 'status', 'approve')">approve</a></li>
<li role="menuitem" ng-if="auth.currentUser.role.indexOf('reviewer') != -1" class="menu-item menu-item-type-post_type menu-item-object-page"><a ng-class="{'hide': result.status == 'not approved'}" ng-click="ctrl.toggleCheck(result, 'status', 'not approve')">not approve</a></li>
+ -->
<li role="menuitem" class="menu-item menu-item-type-post_type menu-item-object-page"><a ng-click="ctrl.openSharedModal(result)">share with</a></li>
<li role="menuitem" ng-if="auth.currentUser.openid == result.owner" class="menu-item menu-item-type-post_type menu-item-object-page"><a ng-click="ctrl.deleteTest(result._id)">delete</a></li>
</ul>
@@ -95,3 +100,4 @@
.button-disabled {
pointer-events:none;
}
+</style>