-
-
Notifications
You must be signed in to change notification settings - Fork 704
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 support #30
Docker support #30
Conversation
@pnmcosta I tested tests and starting Haven't tested Docker but will merge anyway, will test after merging. Thanks for making a lot of improvements, I think they are all great. I will update sections related to env vars and Docker in project's README. |
awesome, glad I could help and learn at the same time :) I've got a microsoft graph oauth implementation on the I'll write up some more details on the wiki, and if you'd like I can also PR it. |
@pnmcosta We already have two ways to authenticate. I suggest just keeping current two: |
Yup, got it, I'll keep the branch cause I'll need it for something i'm working on. |
Please note this PR includes all the changes on #29 as well, so you can accept only this one if you're ok with the other one.
More info about the docker integration here: https://github.com/pnmcosta/saas-boilerplate/wiki/Changes-on-docker-support-branch but the readme as also been updated.