diff --git a/app/controllers/email_smtp_settings_controller.rb b/app/controllers/email_smtp_settings_controller.rb index 6da10172..d41ca570 100644 --- a/app/controllers/email_smtp_settings_controller.rb +++ b/app/controllers/email_smtp_settings_controller.rb @@ -9,7 +9,7 @@ class EmailSmtpSettingsController < ApplicationController def create if @encrypted_config.update(email_configs) - SettingsMailer.smtp_successful_setup(@encrypted_config.value['from_email']).deliver_now! + SettingsMailer.smtp_successful_setup(@encrypted_config.value['from_email'] || current_user.email).deliver_now! redirect_to settings_email_index_path, notice: I18n.t('changes_have_been_saved') else diff --git a/app/mailers/settings_mailer.rb b/app/mailers/settings_mailer.rb index 2aefcf3a..4140ad39 100644 --- a/app/mailers/settings_mailer.rb +++ b/app/mailers/settings_mailer.rb @@ -2,6 +2,6 @@ class SettingsMailer < ApplicationMailer def smtp_successful_setup(email) - mail(to: email, subject: 'SMTP has been configured') + mail(to: email, from: email, subject: 'SMTP has been configured') end end diff --git a/app/views/email_smtp_settings/index.html.erb b/app/views/email_smtp_settings/index.html.erb index 5616f216..ad047b2b 100644 --- a/app/views/email_smtp_settings/index.html.erb +++ b/app/views/email_smtp_settings/index.html.erb @@ -48,7 +48,7 @@
<%= ff.label :from_email, t('send_from_email'), class: 'label' %> - <%= ff.email_field :from_email, value: value['from_email'], required: true, class: 'base-input' %> + <%= ff.email_field :from_email, value: value['from_email'], required: !Docuseal.multitenant?, class: 'base-input' %>
<% end %>
diff --git a/config/routes.rb b/config/routes.rb index 3d10d04d..57f6a830 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -153,9 +153,9 @@ Rails.application.routes.draw do scope '/settings', as: :settings do unless Docuseal.multitenant? resources :storage, only: %i[index create], controller: 'storage_settings' - resources :email, only: %i[index create], controller: 'email_smtp_settings' resources :sms, only: %i[index], controller: 'sms_settings' end + resources :email, only: %i[index create], controller: 'email_smtp_settings' resources :sso, only: %i[index], controller: 'sso_settings' resources :notifications, only: %i[index create], controller: 'notifications_settings' resource :esign, only: %i[show create new update destroy], controller: 'esign_settings'