Skip to content

Re-add mailcatcher to Procfile#1616

Merged
jmhooper merged 1 commit intomasterfrom
jmhooper-fix-mailcatcher
Aug 16, 2017
Merged

Re-add mailcatcher to Procfile#1616
jmhooper merged 1 commit intomasterfrom
jmhooper-fix-mailcatcher

Conversation

@jmhooper
Copy link
Contributor

Why: This allows mailcatcher to be started/stopped alongside the
application. This means that developers won't have to worry about making
sure mailcatcher is running as a daemon while working on the idp.

**Why**: This allows mailcatcher to be started/stopped alongside the
application. This means that developers won't have to worry about making
sure mailcatcher is running as a daemon while working on the idp.
@jmhooper
Copy link
Contributor Author

Before testing this, make sure that mailcatcher is not currently running as a daemon. Visit localhost:1080 and if the mailcatcher UI appears, click Quit to slop mailcatcher. Then make run should start mailcatcher without and problems and ctl+c should shut it down.

@monfresh
Copy link
Contributor

It works after quitting Mailcatcher first. I missed that comment.

Copy link
Contributor

@monfresh monfresh left a comment

Choose a reason for hiding this comment

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

Yay!

@jmhooper jmhooper merged commit a464993 into master Aug 16, 2017
@jmhooper jmhooper deleted the jmhooper-fix-mailcatcher branch December 12, 2017 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants