Actions
Bug #21416
closed
Document mail-releated configuration options
Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Documentation
Target version:
Story points:
-
Release:
Release relationship:
Auto
Description
User asked about "SendUserSetupNotificationEmail" which does not have any comment in the configuration file explaining exactly what it does. I noticed there are a number of other email-related configuration options that also are not documented. We should fill that in.
Updated by Peter Amstutz 10 months ago
- Related to Feature #15397: Declutter the API added
Updated by Peter Amstutz 10 months ago
21416-email-options-doc @ dc333e620d19bde703fddaf39cc8b5a78f0f4f4a
- All agreed upon points are implemented / addressed.
- added & expanded comments to the config defaults file
- Anything not implemented (discovered or discussed during work) has a follow-up story.
- I discovered several keys that are not used. I added them to the list of "decluttering" tasks in the description of #15397
- Code is tested and passing, both automated and manual, what manual testing was done is described
- this is documentation only, so it would be weird to fail tests
- Documentation has been updated.
- yes
- Behaves appropriately at the intended scale (describe intended scale).
- n/a
- Considered backwards and forwards compatibility issues between client and server.
- n/a
- Follows our coding standards and GUI style guidelines.
- yes
Updated by Lucas Di Pentima 10 months ago
Just one observation:
At line 376: It says "Recipients", if more than one address are allowed, I think we should clarify how multiple addresses should be written (comma separated, or in another way?)
The rest LGTM, thanks!
Updated by Peter Amstutz 10 months ago
- Status changed from In Progress to Resolved
Actions