Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2225
Submitted #2261
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2225
Let me check.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2225
@1ambda
after this PR navbar notes list works incorrect.
![peek 2017-04-18
11-40](https://cloud.githubusercontent.com/assets/25951039/25117614/4824e146-242c-11e7-8035-df1e00107aa3
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2225
Merge to master and branch-0.7 if no further comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2225
@1ambda Yeah I also knew it when I was adding the pagination to Helium menu
for both zeppelin.apache.org and Zeppelin :)
Tested again and it works well. LGTM
---
If your project is set up
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2225
@AhyoungRyu thanks for review. didn't know that!
Just updated.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project d
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2225
@1ambda If we update angular-bootstrap version, the pagination in Helium
page won't work since they change the pagination lib name (`pagination` ->`ul
uib-pagination`)
So you might need to
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2225
I was very annoying problem. Thanks @1ambda for the fix.
Tested and LGTM.
Merge to master and branch-0.7 if no further comments.
---
If your project is set up for it, you can reply