added docker exec command line 130 #383
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
NikhilM98
reviewed
Mar 6, 2023
updated port from 8080 to 80 for docker
NikhilM98
requested changes
Mar 6, 2023
| @@ -6,7 +6,6 @@ | |||
|
|
|||
| 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.
| @@ -127,9 +126,14 @@ Sugarizer Server Dashboard is an admin tool for teachers and deployment administ | |||
| * Graphic and request: display graphics and report on previous data. | |||
|
|
|||
| 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.
| 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 | ||
|
|
||
| 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:
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
adding docker exec command in readme.md