Skip to content

Dockerize #211

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

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

Dockerize #211

wants to merge 20 commits into from

Conversation

useffc
Copy link
Contributor

@useffc useffc commented Oct 27, 2018

  • Forum seems to be functional.
  • Hot reloading works

Other Notes

  • Login, signup, create thread, post to thread flows work but I don't have explicit proof of those.

added and read which helps the database get initialized. Should have been between dee8b43 & 4885ab1
@nchase
Copy link
Member

nchase commented Oct 29, 2018

👏 👏 👏

I think the bugs you called out with posts in a thread being duplicated are bugs that currently exist regardless of containerized or not, we just don't see them because all of our threads have tons of posts.

@useffc
Copy link
Contributor Author

useffc commented Oct 29, 2018

disregard the assigning and un-assigning, i'm just click-happy

@useffc
Copy link
Contributor Author

useffc commented Oct 29, 2018

46b88d6 gets tests passing or pending, but I'm a bit concerned about those timeouts.

@useffc useffc mentioned this pull request Oct 30, 2018
Copy link
Member

@nchase nchase left a comment

Choose a reason for hiding this comment

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

👍👍

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.

None yet

2 participants