docs(mail): document SRS forwarding rewrite - #266
Merged
Merged
Conversation
Rephrase the User mailbox section in direct, active voice, and document the Sender Rewriting Scheme (SRS) rewrite that mail forwarding now applies to outgoing envelope senders. Mirror the same changes into the Italian translation. Assisted-by: Claude:claude-sonnet-5
Software Center already blocks a second Mail install on the same node, so replace the outdated warning about multiple instances with a note about port conflicts from other packages or services. Also drop remaining "instance" wording elsewhere in the page, mirrored into the Italian translation. Assisted-by: Claude:claude-sonnet-5
Contributor
|
DavidePrincipi
marked this pull request as draft
September 22, 2026 16:25
DavidePrincipi
marked this pull request as ready for review
September 23, 2026 07:35
DavidePrincipi
force-pushed
the
feat-7741
branch
from
September 23, 2026 07:55
c4e06f1 to
f5f9c1c
Compare
stephdl
requested changes
Sep 23, 2026
Contributor
There was a problem hiding this comment.
SRS content checks out against the running module on a live install. The new note on multiple Mail applications is right too, build-images.sh carries org.nethserver.max-per-node=1.
One sentence is wrong in both languages. Left inline.
Also, the description says Italian was mirrored, but lines 92 and 94 kept the old phrasing while their English counterparts changed.
lucagasparini
approved these changes
Sep 23, 2026
Address review feedback from stephdl on PR #266: - Fix leftover Italian sentences for disabling a mailbox and editing it, to match the direct English phrasing. - Fix the bounce/DSN sentence in both languages: only bounces and DSNs are addressed to the rewritten SRS0 envelope address (never ordinary replies, which follow From:), and the decoded address is the original external sender, not a local mailbox.
stephdl
approved these changes
Sep 23, 2026
Member
Author
|
Merge after Mail 1.9 release... NethServer/dev#7741 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This documents the Sender Rewriting Scheme (SRS) behavior added to mail forwarding, and rewrites the "User mailbox" section to use direct, active language instead of passive/impersonal phrasing. Related to NethServer/dev#7741.
How to test
yarn buildto confirm there are no broken links or invalid MDX.yarn startand review the "Mail" page (English and Italian) for wording and formatting.🤖 Generated with Claude Code