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 planet-installation-vagrant.md #102

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

Conversation

rayyanshaji
Copy link
Contributor

Clarified the installation of planet community on windows, and what url (localhost:3100) to use to access the web application. Feel free to remove the (Windows) text that I added if it doesn't make sense.

Clarified the installation of planet community on windows, and what url (localhost:3100) to use to access the web application
@@ -113,9 +113,11 @@ cd planet
vagrant up prod
```

You now have a working [community Planet](http://localhost:3100) on your OS.
You now have a working [community Planet](http://localhost:3100) on your OS, that can be accessed using [http://localhost:3100](http://localhost:3100).
Copy link
Contributor

Choose a reason for hiding this comment

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

This link is not required as it is already there.

Comment on lines +119 to +120
When the login screen loads, Follow further steps in the [Planet Manual](https://planetlearning.github.io/#!./pages/micromaster/../manual/planet.md) to register as a member.

Copy link
Contributor

Choose a reason for hiding this comment

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

There is another step already which is explaining about configuration of planet community. Planet manual is for using planet after it has been configured

@@ -103,7 +103,7 @@ To solve this problem: Copy link location of Debian 32-bit instead and run comma

---

## Install a Community Planet
## Install a Community Planet (Windows)
Copy link
Contributor

Choose a reason for hiding this comment

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

This is same for all OS so windows is not required

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