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

Changing 'classic notebook' label to 'notebook' inorder to include JN7.0 #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

balajialg
Copy link
Collaborator

@balajialg balajialg commented May 11, 2023

First attempt at tweaking the label to be inclusive of Jupyter Notebook 7.0 release. @yuvipanda - Please feel free to point me in the right direction if I had made any mistakes with this PR. Thanks in advance!

@balajialg
Copy link
Collaborator Author

balajialg commented May 17, 2023

Hi @yuvipanda - We want to test JN 7.0 during a summer course (possibly Data 8) and want the GitHub extension to generate links for launching JN 7.0. Please let me know if this PR is the way forward.

@balajialg
Copy link
Collaborator Author

@yuvipanda Does this PR change the label from "classic notebook" to "notebook" in the extension? If yes, can you merge it as few folks had questions whether classic notebook refers to notebook 7.

@yuvipanda
Copy link
Owner

@balajialg were you able to test this?

@balajialg
Copy link
Collaborator Author

balajialg commented Jun 16, 2023

@yuvipanda I didn't test it. I just realized that you added instructions to test it manually in the README which I missed reading. I will test manually and circle with back with more info. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants