Skip to content
This repository has been archived by the owner on Dec 31, 2020. It is now read-only.

changed 8080 to 80 to fix issue #194 #198

Merged
merged 3 commits into from
Sep 17, 2017

Conversation

saurabh30
Copy link
Contributor

I have just replaced 8080 by 80 everywhere.

@karuppiah7890
Copy link
Contributor

Thanks @saurabh30 for the PR! 😄
And actually, if the HTTP server's port is 80, we don't need to specify it in the URL, so you can just put http://localhost instead of http://localhost:80

Copy link
Contributor

@karuppiah7890 karuppiah7890 left a comment

Choose a reason for hiding this comment

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

Do check the inline comments I put up for small changes :) and then @AndrewGHC can merge this I believe

@@ -22,12 +22,12 @@ After this, you'll need to create your own .env file. Check out the .env.example

Now run `docker-compose up`. This will run all the containers needed to launch this app, and will take some time to finish.

When the process is finished, the app will be exposed on port 8080 and accessible by visiting `http://[hostname]:8080`. You may use a reverse proxy such as Nginx to serve the app on port 80.
When the process is finished, the app will be exposed on port 80 and accessible by visiting `http://[hostname]:80`. You may use a reverse proxy such as Nginx to serve the app on port 80.

This comment was marked as off-topic.

@@ -22,12 +22,12 @@ After this, you'll need to create your own .env file. Check out the .env.example

Now run `docker-compose up`. This will run all the containers needed to launch this app, and will take some time to finish.

When the process is finished, the app will be exposed on port 8080 and accessible by visiting `http://[hostname]:8080`. You may use a reverse proxy such as Nginx to serve the app on port 80.
When the process is finished, the app will be exposed on port 80 and accessible by visiting `http://[hostname]:80`. You may use a reverse proxy such as Nginx to serve the app on port 80.

This comment was marked as off-topic.


###### Installation summary

1. Install and run the Docker daemon.
2. Clone the repository and change into it `git clone https://github.com/freeCodeCamp/Mail-for-Good && cd Mail-for-Good`.
3. Run `cp .env.example .env` then open `.env` and edit it, passing in your own values.
4. Run `docker-compose up`. Wait for it to finish.
5. Visit `http://[hostname]:8080` or `localhost:8080` if running locally.
5. Visit `http://[hostname]:80` or `localhost` if running locally.

This comment was marked as off-topic.

@zhakk-harn
Copy link
Contributor

It also seems good to me.
@saurabh30, do you mind if I squash these commits when merging them?

@karuppiah7890
Copy link
Contributor

Same here, LGTM! :) Thanks @saurabh30

@AndrewWalsh
Copy link
Contributor

Lgtm, cheers

@AndrewWalsh AndrewWalsh merged commit c827565 into freeCodeCamp:master Sep 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants