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

Questions... #26

Open
jasonwhitedev opened this issue Oct 26, 2019 · 1 comment
Open

Questions... #26

jasonwhitedev opened this issue Oct 26, 2019 · 1 comment

Comments

@jasonwhitedev
Copy link

Just two quick questions.

Why is line 12 of docker-compose.yml this:

  • flask-app-db:/var/lib/postgresql/data
    instead of this:
  • flask-app-db:/var/lib/docker/volumes/flask-app-db/_data
    On Ubuntu using the former means that db data does not persist when the server is restarted.

Why is line 58 of api/views/main.py
message=f"Successfully created person {new_person.name} with id: {new_person._id}" and not
message=f"Successfully created person {new_person.name} with id: {new_person.id}"
Is this just a typo? - new_person._id - new_person.id ???

@tko22
Copy link
Owner

tko22 commented Jan 11, 2021

  1. That's weird that it doesnt work on ubuntu. The syntax for docker compose when linking a volume is docker volume name: folder to mount within docker container". In this case var/lib/postgresql/data` is where postgres data is stored
  2. a new fix is out for that. It's the same

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

No branches or pull requests

2 participants