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

nbclassic to use its own static assets #96

Merged
merged 27 commits into from
May 11, 2022

Conversation

echarles
Copy link
Member

@echarles echarles commented May 4, 2022

In jupyter/notebook-team-compass#5 (comment) we have defined that nbclassic should have its own assets.

Further to #93 and #94, this PR ensure after those commands, the UI is shown and operational.

pip install -e .
jupyter nbclassic

Before merging, a minimal set of UI tests should be operational.

  • pip install -e .
  • jupyter nbclassic
  • minimal CI (remaining minor failures can be fixed in subsequent PRs)

docs/source/links.txt Outdated Show resolved Hide resolved
docs/source/links.txt Outdated Show resolved Hide resolved
docs/source/notebook.rst Outdated Show resolved Hide resolved
docs/source/template.tpl Outdated Show resolved Hide resolved
echarles and others added 6 commits May 5, 2022 07:26
Co-authored-by: Rosio <rosio_reyes@yahoo.com>
Co-authored-by: Rosio <rosio_reyes@yahoo.com>
Co-authored-by: Rosio <rosio_reyes@yahoo.com>
Co-authored-by: Rosio <rosio_reyes@yahoo.com>
Co-authored-by: Rosio <rosio_reyes@yahoo.com>
…pynb

Co-authored-by: Rosio <rosio_reyes@yahoo.com>
@echarles echarles added the enhancement New feature or request label May 10, 2022
@echarles echarles marked this pull request as ready for review May 10, 2022 16:18
@echarles
Copy link
Member Author

@Zsailer I have removed __version__.py and tested with 3 classic extensions which all work fine nbgrader, rise and nbdime.

Screenshot 2022-05-11 at 18 10 38

Screenshot 2022-05-11 at 18 09 23

This is ready for review/merge.

@echarles
Copy link
Member Author

more screenshots of the extensions working after launching jupyter nbclassic.

Screenshot 2022-05-11 at 18 18 22

Screenshot 2022-05-11 at 18 18 38

@Zsailer
Copy link
Member

Zsailer commented May 11, 2022

Looks good to me, thanks @echarles!

@echarles echarles merged commit 5328e00 into jupyter:main May 11, 2022
@echarles
Copy link
Member Author

Thx @Zsailer I have merged. We will iterate to further fix the CI and make soon a release 👍

@minrk minrk mentioned this pull request Jul 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants