Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/883
---
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 the fea
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/883#issuecomment-219919689
Merge it into master if there're no more discussions
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/883#issuecomment-219578496
LGTM
---
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 fe
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/883#issuecomment-218983138
@Leemoonsoo Just added test. Please review
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/883#issuecomment-218774255
Thanks for the fix. How about add a test case to
https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-web/test/spec/factory/notebookList.js#L1
GitHub user minahlee opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/883
Fix notebook folder structure name match bug
### What is this PR for?
Current regex match rule for notebook folder takes not only `/` as
delimiter but also `[` and `]`.
###
