Skip to content

Commit 88c5338

Browse files
committed
Implement Thomas Kluyver's idea -- let the contents and kernel manager get the application option as part of calculating the default
1 parent 8880ccb commit 88c5338

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

jupyter_server/serverapp.py

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1149,14 +1149,6 @@ def _root_dir_validate(self, proposal):
11491149
raise TraitError(trans.gettext("No such notebook dir: '%r'") % value)
11501150
return value
11511151

1152-
@observe('root_dir')
1153-
def _update_root_dir(self, change):
1154-
"""Do a bit of validation of the notebook dir."""
1155-
# setting App.root_dir implies setting notebook and kernel dirs as well
1156-
new = change['new']
1157-
self.config.FileContentsManager.root_dir = new
1158-
self.config.MappingKernelManager.root_dir = new
1159-
11601152
@observe('server_extensions')
11611153
def _update_server_extensions(self, change):
11621154
self.log.warning(_("server_extensions is deprecated, use jpserver_extensions"))

0 commit comments

Comments
 (0)