[FEATURE] Option to Disable Mail

[jonas-hoebenreich]

Is your feature request related to a problem? Please describe.

We do not have Mail configured and do not wish to configure it as we do not see the additional value. Currently, our logs are getting spammed with messages like: Could not Email :: SMTP not configured :: [...] This clutter makes it harder to identify other important log messages.

Describe the solution you'd like

Introduce a new environment variable, DISABLE_MAIL. When this variable is set:

• The notification settings related to email are disabled in the frontend.
• The application does not attempt to send emails.

Benefits:

• Cleaner logs without unnecessary error messages.
• A more user-friendly configuration for teams not using the mail feature.

Describe alternatives you've considered

No response

Additional context

No response

[devsargam]

Hey if this issue is valid can I work on it @jonas-hoebenreich?

[mattinannt]

@jonas-hoebenreich makes sense.

@devsargam Thanks for offering your help :-) I will assign you to the issue :-) We already have a IS_SMTP_CONFIGURED variable in packages/email/index.tsx that could be used for these checks. We just need to make sure in the sendEmail function that we don't print the error. It should be an easy fix :-)

[devsargam]

Hey @mattinannt, what is the ideal thing to do?

Should I remove the else part? or is there some other desired result?

[jonas-hoebenreich]

I personally would just remove the else clause. And it would be great to have a notice in the notification settings:

[devsargam]

Thanks @jonas-hoebenreich this helps raising a pr with the fix you wanted :)

[mattinannt]

@devsargam How is the development going. When do you think you will be able to open a PR for it? :-)

[devsargam]

Hey @mattinannt was waiting for your input? Is the approach discussed correct?

[mattinannt]

@devsargam Yes, just deleting the else case should be the way to go :-) Thanks for raising a PR 💪🚀