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

Update README.md #7474

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update README.md #7474

wants to merge 2 commits into from

Conversation

hackit-coder
Copy link

Steps to host Jpyter as web interface on Google, Mozilla etc.

Steps to host Jpyter as web interface on Google, Mozilla etc.
Copy link
Contributor

github-actions bot commented Oct 3, 2024

Binder 👈 Launch a Binder on branch hackit-coder/notebook/patch-2

@hackit-coder
Copy link
Author

Hello anyone there

Copy link
Member

@krassowski krassowski left a comment

Choose a reason for hiding this comment

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

These instructions are incorrect for a number of reasons:

  • the port 8888 may be taken and a different port would be used instead
  • usually there is no need to open a browser - the command above will open the default browser
  • localhost may not work on specific systems and instead using 127.0.0.1 may be needed
  • the URL above will not include the token needed to login and this token will be different each time

Each PR should address an issue. This PR is not linked to any issue. In future I would encourage you to:

  • open an issue first to discuss potential enhancements
  • if you are looking for a way to contribute, looking through the issues and picking up something tagged as good first issue :)

@hackit-coder
Copy link
Author

Ok changed it a bit

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