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

Add SMTP email config vars back in #400

Merged
merged 2 commits into from
May 1, 2020
Merged

Add SMTP email config vars back in #400

merged 2 commits into from
May 1, 2020

Conversation

patfreeman
Copy link
Contributor

No description provided.

@mik3y
Copy link
Member

mik3y commented Apr 24, 2020

Ah, thanks for fixing this! I knew I broke something.

What do you think about pulling in dj-email-url and make this one config var instead? eg KEGBOT_EMAIL_CONFIG=smtp://user:password@host/ .. trying to make configuration less of an ordeal, might be a little easier.

@patfreeman
Copy link
Contributor Author

@mik3y I'm not sure what you want to do about 1aa8df2. The updates to kegbot-api broke gunicorn in server.

@mik3y
Copy link
Member

mik3y commented Apr 30, 2020

@patfreeman doh! let me see about getting that fixed

@mik3y mik3y force-pushed the email branch 2 times, most recently from 8132441 to 2290130 Compare May 1, 2020 02:17
@mik3y mik3y merged commit e0ee09c into Kegbot:master May 1, 2020
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