Improve server screen UI#5152
Conversation
mtrutledge
left a comment
There was a problem hiding this comment.
Hey @donker, wouldn't we need to do the same changes on
/Dnn.PersonaBar.Extensions/Services/ServerSettingsSmtpAdminController.cs
that you did on
/Dnn.PersonaBar.Extensions/Services/ServerSettingsSmtpHostController.cs
since the SMTP service on the javascript end checks if the user is an admin or a host and then calls a different end point?
Good catch. Thanks. |
mtrutledge
left a comment
There was a problem hiding this comment.
This looks good to me. Thanks @donker
|
Good job @donker - thanks for the quick contribution on this! |
Fixed an issue that prevented sending smtp test emails Due to the UI changes in dnnsoftware#5152 the frontend now posts an obfuscated password unless the user is currently changing the password. That scenario got handled correctly for UpdateSmtpSettings but the same logic needed to be implements in SendTestEmail for that button to work fine.
Fixed an issue that prevented sending smtp test emails Due to the UI changes in #5152 the frontend now posts an obfuscated password unless the user is currently changing the password. That scenario got handled correctly for UpdateSmtpSettings but the same logic needed to be implements in SendTestEmail for that button to work fine.
Fixed an issue that prevented sending smtp test emails Due to the UI changes in dnnsoftware#5152 the frontend now posts an obfuscated password unless the user is currently changing the password. That scenario got handled correctly for UpdateSmtpSettings but the same logic needed to be implements in SendTestEmail for that button to work fine.
Improves the edit boxes UI
