Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JupyterLab 4 support #1343

Merged
merged 20 commits into from
Jul 6, 2023
Merged

JupyterLab 4 support #1343

merged 20 commits into from
Jul 6, 2023

Conversation

trungleduc
Copy link
Member

@trungleduc trungleduc commented Jun 28, 2023

References

#1304

Code changes

  • Update to JupyterLab 4 components
  • Mock the service manager to remove unused services.
  • Create a theme manager to load JupyterLab themes without depending on the setting plugin.
  • Tree page is now a Lab remix app.

User-facing changes

  • New configurations in VoilaConfiguration:
    • VoilaConfiguration.extension_whitelist: The list of enabled JupyterLab extensions, if None, all extensions are loaded.
      This setting has higher priority than the extension_blacklist
    • VoilaConfiguration.extension_blacklist: The list of disabled JupyterLab extensions, if None, all extensions are loaded

Backwards-incompatible changes

  • Theme and template config are renamed:

    • voila-theme -> theme
    • voila-template -> template
  • Theme argument now is the theme name displayed in JupyterLab and not the name of the theme python package.

@github-actions
Copy link
Contributor

Binder 👈 Launch a Binder on branch trungleduc/voila/lab4

voila/configuration.py Outdated Show resolved Hide resolved
voila/configuration.py Outdated Show resolved Hide resolved
packages/voila/src/voilaplugins/path.ts Outdated Show resolved Hide resolved
voila/tornado/treehandler.py Outdated Show resolved Hide resolved
voila/utils.py Outdated Show resolved Hide resolved
@trungleduc trungleduc marked this pull request as ready for review July 6, 2023 12:58
packages/voila/src/app.ts Outdated Show resolved Hide resolved
packages/voila/src/tools.ts Outdated Show resolved Hide resolved
voila/handler.py Show resolved Hide resolved
@martinRenou martinRenou merged commit ea11d94 into voila-dashboards:main Jul 6, 2023
8 of 10 checks passed
@martinRenou
Copy link
Member

Thank you !!

@SylvainCorlay
Copy link
Member

Yeah!

@@ -140,3 +140,19 @@ class VoilaConfiguration(traitlets.config.Configurable):
help="""Size of pre-heated kernel pool for each notebook. Zero or negative number means disabled.
""",
)

extension_whitelist = List(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe 0.5.0 could be a good time to switch to allowlist and denylist?

Since it was also suggested in #725.

"@voila-dashboards/jupyterlab-preview",
"@jupyter/collaboration-extension",
]
must_have_extensions = ["@jupyter-widgets/jupyterlab-manager"]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe required_extensions would also work as an alternative to must_have_extensions?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-change dependencies Pull requests that update a dependency file enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants