Skip to content

Added 'clear' messages as a configurable option - #447

Open
metju90 wants to merge 1 commit into
sj26:mainfrom
metju90:master
Open

Added 'clear' messages as a configurable option#447
metju90 wants to merge 1 commit into
sj26:mainfrom
metju90:master

Conversation

@metju90

@metju90 metju90 commented Feb 24, 2021

Copy link
Copy Markdown

We planned to persist the emails caught by this software but unfortunately one colleague was not aware of this and deleted all emails by pressing "clear".

This motivated me to open this PR, to have the "clear" button as a config option.

@metju90 metju90 changed the title Made 'clear' messages as a configurable option Added 'clear' messages as a configurable option Apr 5, 2021
@sj26

sj26 commented Jul 15, 2021

Copy link
Copy Markdown
Owner

MailCatcher should not be used as a durable store for messages, it uses an in-memory database, so I'm not sure I'm keen on adding this as an option. What's your use case here; why don't you want emails to be cleared?

@damienfinck

Copy link
Copy Markdown

For my case: we use mailcatcher for recipe environments where many users have access. So it happens that users click on it and delete everyone's mails. So I removed the "clear" button and put a limit to keep only the last 300 messages.
We use mailcatcher with persistent storage in Docker containers.

@sj26 sj26 linked an issue Nov 1, 2021 that may be closed by this pull request
@shub-qantas

Copy link
Copy Markdown

@damienfinck how did you remove the clear button?

@damienfinck

Copy link
Copy Markdown

@shub-qantas I took the code from the master branch, applied this PR and regenerated the mailcatcher application in a Docker container so I can deploy it anywhere.

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.

Add option to remove "clear" functionality from UI

4 participants