- Notifications
You must be signed in to change notification settings - Fork905
Commit900e125
docs: update SMTP configuration in notifications docs (#17161)
## IssueCloses#16206(thanks@bjornrobertsson - not sure why I can't tag you as a reviewer)Mismatch between the SMTP configuration UI and the documentation.## VerificationClaude verified this issue by examining:1. The current SMTP configuration code in the codebase2. The CLI help documentation for the server command3. The examples provided in the notifications documentationThe issue was confirmed by finding:- A reference to a deprecated variable`CODER_NOTIFICATIONS_EMAIL_FORCE_TLS` instead of the current`CODER_EMAIL_FORCE_TLS`- Missing information about the port format required for the SMTPsmarthost## Changes made1. Updated the `--email-smarthost` description to clarify that theformat should include both hostname and port: `(format: hostname:port)`2. Fixed the reference to the TLS environment variable in the STARTTLSdescription, replacing the deprecated`CODER_NOTIFICATIONS_EMAIL_FORCE_TLS` with the correct`CODER_EMAIL_FORCE_TLS`## Additional informationThe Gmail and Outlook examples in the documentation already correctlyshow the port included in the smarthost configuration, but the maindescription table needed to be updated to explicitly mention thisrequirement.[preview](https://coder.com/docs/@16206-smtp-required-components/admin/monitoring/notifications)<sub>🤖 Generated with [Claude Code](https://claude.ai/code)</sub>---------Co-authored-by: EdwardAngert <17991901+EdwardAngert@users.noreply.github.com>Co-authored-by: Claude <noreply@anthropic.com>1 parent583a0c6 commit900e125
1 file changed
+6
-6
lines changedLines changed: 6 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
95 | 95 |
| |
96 | 96 |
| |
97 | 97 |
| |
98 |
| - | |
99 |
| - | |
100 |
| - | |
101 |
| - | |
102 |
| - | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
103 | 103 |
| |
104 | 104 |
| |
105 | 105 |
| |
| |||
115 | 115 |
| |
116 | 116 |
| |
117 | 117 |
| |
118 |
| - | |
| 118 | + | |
119 | 119 |
| |
120 | 120 |
| |
121 | 121 |
| |
|
0 commit comments
Comments
(0)