[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4624: [ZEPPELIN-5934] Check notebook folder permissions before allowing to rename, remove or restore it

2023-06-27 Thread via GitHub
jongyoul commented on code in PR #4624: URL: https://github.com/apache/zeppelin/pull/4624#discussion_r1244500227 ## zeppelin-server/src/main/java/org/apache/zeppelin/service/NotebookService.java: ## @@ -1497,4 +1530,78 @@ private boolean checkPermission(String noteId,

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4624: [ZEPPELIN-5934] Check notebook folder permissions before allowing to rename, remove or restore it

2023-06-27 Thread via GitHub
jongyoul commented on code in PR #4624: URL: https://github.com/apache/zeppelin/pull/4624#discussion_r1244498036 ## zeppelin-server/src/main/java/org/apache/zeppelin/service/NotebookService.java: ## @@ -1192,24 +1203,38 @@ public void moveFolderToTrash(String folderPath,