Repository navigation
fix(ui): select the matching sender for replies - #13802
LorenzLoesch wants to merge 1 commit into
Conversation
| <NcFormBoxSwitch | ||
| :model-value="replyFromMatchingAddress" | ||
| :label="t('mail', 'Automatically select the sender address for replies')" | ||
| :description="t('mail', 'Use the account or alias involved in the message')" |
There was a problem hiding this comment.
@kra-mo can we make it the default without a option to change? It sound reasonable to me to automatically pick the recipient as alias (if defined).
There was a problem hiding this comment.
Yeah, people can always manually change it on a case-by-case basis if needed.
There was a problem hiding this comment.
sure, do you want me to change it that way?
i modelled the feature after gmail, where you can change it in the settings. but i agree that this is not necessary.
There was a problem hiding this comment.
I modelled the feature after gmail
Yeah, maybe let's not use this as a reference for design ;)
There was a problem hiding this comment.
i think the pr is in good shape now. let me know if anything else needs to change.
bb2e045 to
714cbfe
Compare
Replies to alias-addressed mail defaulted to the account primary, which changed the sending identity in an existing conversation. Match the clicked message against configured accounts and aliases; the composer provides a per-message override. Preserve outgoing recipients independently of the selected identity and clear the old alias when choosing a primary address. Identity inference is limited to the clicked message. New messages, forwards, and restored drafts retain their existing identity handling. Signed-off-by: Lorenz Loesch <lorenz.loesch@gmail.com>
31da7da to
55fbb2e
Compare
This branch has not been deployed
3 participants
Replies to mail received through an alias currently default to the account's primary address. Automatically select the configured account or alias involved in the clicked message, including identities on other connected accounts. Users can override the From address in the composer for each message.
Fixes #6236. Related: #607 and #10559 (closed duplicate of #6236).
Validation:
composer test:unit -- --no-coverage). The suite emits three existing discovery warnings for missing S/MIME chain-key fixtures.AI assistance: Implementation and local reviews were assisted by Codex (GPT-6).