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

Added note for fixing PostgreSQL issue #27

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

Conversation

dokun1
Copy link

@dokun1 dokun1 commented Mar 27, 2020

Hi Tim!!! 👋 Congratulations on your release!

I tried the sample, and I ran into an issue with PostgreSQL that ended up being easy to solve, but wasn't super obvious to fix. For those that may be confused by the issue, I've done the following:

  1. I added a stopDB.sh script which will cut off the existing container you create with setupDB.sh.
  2. I added to your README.md file with instructions on how to see if PostgreSQL is already running.
  3. I added to your README.md file with one example of how to remove it.

I also went ahead and put all your bash scripts into a bin directory for the sake of organization. Thank you for making this!

swift run
```

This will create a site at http://localhost:8080. The blog can be found at http://localhost:8080/blog/ and you can login at http://localhost:8080/blog/admin/. The first time you visit the login a user will be created and the details printed to the console.

**Note**: If you receive an error such as "Thread 1: Fatal error: Error raised at top level...", then it is likely you are already running an instance of PostgreSQL on your machine. To try and resolve:
Copy link
Member

Choose a reason for hiding this comment

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

This could be due to a number of errors, such as something already running on port 8080 so we probably need to quality this better

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