Skip to content

Update README.md#148

Closed
smyja wants to merge 1 commit intoaxllent:developfrom
smyja:patch-1
Closed

Update README.md#148
smyja wants to merge 1 commit intoaxllent:developfrom
smyja:patch-1

Conversation

@smyja
Copy link
Copy Markdown

@smyja smyja commented Aug 3, 2023

No description provided.

@axllent
Copy link
Copy Markdown
Owner

axllent commented Aug 3, 2023

I must be missing something. Could you please explain the relevance of a) undoing a change I made yesterday regarding notifications, and b) adding the most simplistic "how to run Mailpit" to just the brew instructions?

Regarding "b" above, was it not clear to you how to run it?

@smyja
Copy link
Copy Markdown
Author

smyja commented Aug 3, 2023

I must be missing something. Could you please explain the relevance of a) undoing a change I made yesterday regarding notifications, and b) adding the most simplistic "how to run Mailpit" to just the brew instructions?

Regarding "b" above, was it not clear to you how to run it?

Actually it wasn't clear but you can delete/close this. Sorry about A, that was a mistake. I have no clue how that happened.

@axllent
Copy link
Copy Markdown
Owner

axllent commented Aug 3, 2023

Thanks for the pointer about it not being clear, feedback is great. I will close this, however (soon) I'll add a new general section about "Running Mailpit" which will apply to all installation methods. Thanks again.

@axllent axllent closed this Aug 3, 2023
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