winter icon indicating copy to clipboard operation
winter copied to clipboard

Add failover transport to MailSetting

Open mjauvin opened this issue 10 months ago • 4 comments

image

mjauvin avatar Feb 23 '25 18:02 mjauvin

@mjauvin are people able to configure the transports through the backend as well as use the failover transport?

LukeTowers avatar Feb 25 '25 22:02 LukeTowers

@mjauvin are people able to configure the transports through the backend as well as use the failover transport?

No, they would need to configure the other mailers in the config

mjauvin avatar Feb 25 '25 22:02 mjauvin

As I wrote in discord:

<@164272636988227584> I'm thinking we should move away from the SettingsModel behavior for the MailSetting model and have an active record instead (for the default mailer). That would allow setting multiple records for the different transports to support the failover (and roundrobin in Laravel 11)

mjauvin avatar Feb 25 '25 22:02 mjauvin

@mjauvin are people able to configure the transports through the backend as well as use the failover transport?

No, they would need to configure the other mailers in the config

All mailers can be configured through the backend as usual, and they are all saved together. Therefore, in theory, you should first configure the individual mailers, then select and configure the failover, and finally save everything in a single action.

From a user perspective, it would probably be more intuitive if the configurations of the selected failover mailers were visible, preventing them from being "hidden."

matteotrubini avatar Feb 26 '25 07:02 matteotrubini