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

Docker compose - local development fixes #51

Closed
wants to merge 4 commits into from
Closed

Docker compose - local development fixes #51

wants to merge 4 commits into from

Conversation

tommilligan
Copy link

@tommilligan tommilligan commented Apr 8, 2018

I'm interested in the project, and thinking of taking on one of the open issues. I ran into a few issues getting local development with docker-compose set up - this PR suggests some fixes:

  • incompatible with mongo:latest
    • changed docker compose to use mongo:3.4
  • npm install -q fails to install bower modules.
    • use yarn and and copy bower config files to the build container
  • cannot debug sent emails
    • added a mail service based on maildev. This lightweight container allows UI debugging of emails during development
  • always regenerate configuration when running Dockerfile

Feedback welcome, as well as any thoughts you have on the project roadmap?

@tommilligan
Copy link
Author

@lukevella is this something you're still interested in?

@Avalarion
Copy link

Hey Tom,

if @lukevella is not responding, would you create a fork? There are a lot of features that I would love to see, but I am not able to write NodeJS =/.

This here is more or less one...

Have a nice weekend,

Bastian

@tommilligan
Copy link
Author

Thanks @Avalarion, I might do if I have time. Fraid' I can't promise anything at the moment though.

If you have some thoughts on features you'd like to try and implement yourself, I'm happy to get you started on the right track/give you some pointers etc.

@tommilligan
Copy link
Author

Closing this as the project looks dead (16 months with no reply from maintainer, 8 months since last commit).

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