zeppelin-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From b..@apache.org
Subject zeppelin git commit: [ZEPPELIN-1122] BugFix for repositories snapshot is always set to true.
Date Mon, 11 Jul 2016 02:04:26 GMT
Repository: zeppelin
Updated Branches:
  refs/heads/master 6a82e7aab -> 60186d8bb


[ZEPPELIN-1122] BugFix for repositories snapshot is always set to true.

### What is this PR for?
This PR fixes a bug about snapshot setting and select box(true,false) html.

### What type of PR is it?
Bug Fix | Improvement

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1122

### How should this be tested?
1. go Interpreter menu.
2. click ```repository information``` button that is right top.
3. click plus button (to Add New Repository)
and refer to screenshot.

### Screenshots (if appropriate)
- before
![b](https://cloud.githubusercontent.com/assets/3348133/16590775/e5476c82-4313-11e6-8abf-a1bb662b6ae4.gif)

- after
![a](https://cloud.githubusercontent.com/assets/3348133/16590780/ea491ce4-4313-11e6-82f0-29d7cd9cb0a0.gif)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: astroshim <hsshim@nflabs.com>

Closes #1137 from astroshim/ZEPPELIN-1122 and squashes the following commits:

0ce28a0 [astroshim] rebase
be684c8 [astroshim] remove snapshot from popup
829e9c1 [astroshim] fix select box and snapshot backend code.


Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/60186d8b
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/60186d8b
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/60186d8b

Branch: refs/heads/master
Commit: 60186d8bb9c3b4ccfc63f1326400618fd59a5d02
Parents: 6a82e7a
Author: astroshim <hsshim@nflabs.com>
Authored: Sun Jul 10 20:51:29 2016 +0900
Committer: Alexander Bezzubov <bzz@apache.org>
Committed: Mon Jul 11 11:04:21 2016 +0900

----------------------------------------------------------------------
 zeppelin-web/src/app/interpreter/interpreter.html        | 11 ++++-------
 .../components/repository-create/repository-dialog.html  |  7 ++++---
 2 files changed, 8 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zeppelin/blob/60186d8b/zeppelin-web/src/app/interpreter/interpreter.html
----------------------------------------------------------------------
diff --git a/zeppelin-web/src/app/interpreter/interpreter.html b/zeppelin-web/src/app/interpreter/interpreter.html
index af3b844..d96fadf 100644
--- a/zeppelin-web/src/app/interpreter/interpreter.html
+++ b/zeppelin-web/src/app/interpreter/interpreter.html
@@ -58,17 +58,14 @@ limitations under the License.
           <a tabindex="0" class="btn btn-info" role="button"
              popover-trigger="focus"
              popover-placement="right"
-             popover-html-unsafe="
-               <label>URL: </label>
-               {{repo.url}}<br/>
-               <label>Snapshot: </label>
-               {{repo.snapshotPolicy.enabled}}<br/>
+             popover-html-unsafe="<label>URL: </label>
+               {{repo.url}}<br>
                <label>Username: </label>
                {{repo.authentication.username}}">
             <span class="fa fa-database"></span>
             {{repo.id}}&nbsp;
-            <span ng-if="!isDefaultRepository(repo.id)"
-                  class="fa fa-close blackOpc" ng-click="removeRepository(repo.id)"></span>
+            <span ng-if="!isDefaultRepository(repo.id)" class="fa fa-close blackOpc"
+                  ng-click="removeRepository(repo.id)"></span>
           </a>
         </li>
         <li class="liVertical">

http://git-wip-us.apache.org/repos/asf/zeppelin/blob/60186d8b/zeppelin-web/src/components/repository-create/repository-dialog.html
----------------------------------------------------------------------
diff --git a/zeppelin-web/src/components/repository-create/repository-dialog.html b/zeppelin-web/src/components/repository-create/repository-dialog.html
index 1bc7b07..d2efec3 100644
--- a/zeppelin-web/src/components/repository-create/repository-dialog.html
+++ b/zeppelin-web/src/components/repository-create/repository-dialog.html
@@ -43,9 +43,10 @@ limitations under the License.
             <div class="form-group">
               <label class="control-label col-sm-2" for="repoSnapshot">Snapshot</label>
               <div class="col-sm-10">
-                <select class="form-control" id="repoSnapshot" ng-model="newRepoSetting.snapshot">
-                  <option ng-selected="true">false</option>
-                  <option>true</option>
+                <select class="form-control"
+                        id="repoSnapshot"
+                        ng-model="newRepoSetting.snapshot"
+                        ng-options="col for col in [false,true]">
                 </select>
               </div>
             </div>


Mime
View raw message