Add optional (opt-in) support for Mailpit #779
Conversation
|
@navarr This goes for this discussion: https://github.com/orgs/wardenenv/discussions/780 |
There was a problem hiding this comment.
The tags in this file don't use the "array of strings" format because the redirect wasn't working unless I used the object style notation.
|
@navarr This is ready for your review. |
| @@ -0,0 +1,27 @@ | |||
| services: | |||
| mailpit: | |||
| container_name: mailhog | |||
There was a problem hiding this comment.
why the container name is mailhog?
Just During upgrade, it caused a conflict
There was a problem hiding this comment.
For backwards compatibility so you can still reference "mailhog" but it's really just mailpit. This can easily be removed and whatever breaks will need to be upgraded.
There was a problem hiding this comment.
The simple solution would be: warden svc down --remove-orphans && warden svc up.
There was a problem hiding this comment.
Yeah, I did like that and it worked. But I got confused that container is named mailhog, while we don't have such service and domain. So I would prefer removing this line
Adds ability to easily switch to Mailpit, a maintained successor to Mailhog.