passbolt / passbolt_api

Passbolt Community Edition (CE) API. The JSON API for the open source password manager for teams!
https://passbolt.com
GNU Affero General Public License v3.0
4.63k stars 305 forks source link

"Email server > Settings" buttons disabled but no reason given #456

Open garrettboone opened 1 year ago

garrettboone commented 1 year ago

"Email server > Settings" buttons disabled but no reason given

What you did

Installed via source, provided SMTP info via install script, went to send a test email.

What happened

"Send test email" button was disabled. No errors.

What you expected to happen

A note similar to the one that shows when any fields are modified. The note says: Warning: These are the settings provided by a configuration file. If you save it, will ignore the settings on file and use the ones from the database.

A note with similiar might be able to always show. Something like Warning: These settings are provided by a configuration file. If any changes are made on this screen, passbolt will ignore the settings in the file and add them to encrypted organization settings in the database.

I actually checked for errors as a first step as it wasn't clear why it was disabled.

garrettboone commented 1 year ago

Also, once the settings are updated in the database, the warning about settings sourced from file is still showing. It shows because the file still has settings, yes - but they are no longer used. Maybe the warning above that is proposed would address this as well.