added docker exec command line 130#383
Conversation
updated port from 8080 to 80 for docker
NikhilM98
left a comment
NikhilM98
left a comment
There was a problem hiding this comment.
The destination branch for your PR should be dev, not master.
|
|
||
| Sugarizer Server allows the deployment of Sugarizer on a local server, for example on a school server, so expose locally Sugarizer as a Web Application. Sugarizer Server can also be used to provide collaboration features for Sugarizer Application on the network. Sugarizer Server could be deployed in a Docker container or on any computer with Node.js 10+ and MongoDB 3.2+. | ||
|
|
||
|
|
There was a problem hiding this comment.
Remove unnecessary changes.
|
|
||
| To login to the Dashboard the first time, you will have to create an admin account using this command: | ||
|
|
||
There was a problem hiding this comment.
There is some whitespace here.
|
|
||
| Incase using Docker to login the Dashboard for the first time | ||
|
|
||
| docker exec -it <id of sugarizer-server_server> bash |
There was a problem hiding this comment.
We can also consider using docker-compose exec server bash instead of this command. That way we don't have to worry about the ID of the container. @nitrophenol can you test that?
@llaske what is your opinion on using docker-compose exec instead?
| sh add-admin.sh admin password http://127.0.0.1:8080/auth/signup | ||
|
|
||
| Incase using Docker to login the Dashboard for the first time |
There was a problem hiding this comment.
Maybe replace Incase using Docker to login the Dashboard for the first time with something like If Sugarizer Server was deployed using Docker Compose, use these commands to create an admin account:
2 participants
adding docker exec command in readme.md