Skip to content

feat: Allow user-level sender addresses via mail provided - #64395

Open
ahujadivyam wants to merge 1 commit into
nextcloud:masterfrom
ahujadivyam:feat/56904-sharebymail-user-sender-address
Open

ahujadivyam wants to merge 1 commit into
nextcloud:masterfrom
ahujadivyam:feat/56904-sharebymail-user-sender-address

Conversation

@ahujadivyam

@ahujadivyam ahujadivyam commented Sep 15, 2026

Copy link
Copy Markdown

This commit updates the ShareByMailProvider to support sending share emails using the user's personal email address via the Mail Provider API, mirroring the functionality available for Calendar invitations (IMipPlugin).

  • Added an admin setting ('useUserEmail') to enable/disable user-level senders.
  • Modified ShareByMailProvider to try to find an appropriate Mail Provider service.
  • If a Mail Provider is available and user-level sender is enabled, emails are sent from the sharing user's address.
  • If no provider is available, or an error occurs during sending, it cleanly falls back to the existing system mailer logic.
  • Updated relevant UI components and unit tests.

Summary

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@ahujadivyam
ahujadivyam requested review from a team as code owners September 15, 2026 21:21
@ahujadivyam
ahujadivyam requested review from Altahrim, come-nc, icewind1991, kristian-zendato, skjnldsv and sorbaugh and removed request for a team September 15, 2026 21:21
@ahujadivyam ahujadivyam changed the title Enhance sharebymail: Allow user-level sender addresses via Mail Provi… feat: Allow user-level sender addresses via mail provided Sep 15, 2026
@ahujadivyam
ahujadivyam force-pushed the feat/56904-sharebymail-user-sender-address branch from f6524af to fe16c29 Compare September 21, 2026 10:51
@ahujadivyam
ahujadivyam requested a review from a team as a code owner September 21, 2026 10:51
@ahujadivyam
ahujadivyam force-pushed the feat/56904-sharebymail-user-sender-address branch from cd51e57 to 1ee80b7 Compare September 21, 2026 11:02
@miaulalala miaulalala added this to the Nextcloud 36 milestone Sep 21, 2026
@miaulalala miaulalala added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Sep 21, 2026
@ahujadivyam
ahujadivyam force-pushed the feat/56904-sharebymail-user-sender-address branch 7 times, most recently from ba345e3 to 3817c3f Compare September 21, 2026 14:34
@ahujadivyam

Copy link
Copy Markdown
Author

Hi @miaulalala , I fixed some of the errors. Can you help me with approvals for running the workflows.
Thanks!

…r API

Resolves nextcloud#56904

This commit updates the ShareByMailProvider to support sending share emails
using the user's personal email address via the Mail Provider API, mirroring
the functionality available for Calendar invitations (IMipPlugin).

- Added an admin setting ('useUserEmail') to enable/disable user-level senders.
- Modified ShareByMailProvider to try to find an appropriate Mail Provider service.
- If a Mail Provider is available and user-level sender is enabled, emails are
  sent from the sharing user's address.
- If no provider is available, or an error occurs during sending, it cleanly
  falls back to the existing system mailer logic.
- Updated relevant UI components and unit tests.

Signed-off-by: Divyam <divyam@divyam.dev>
@ahujadivyam
ahujadivyam force-pushed the feat/56904-sharebymail-user-sender-address branch from e65a2cb to db23591 Compare September 22, 2026 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow user-level sender addresses for “Share by mail” (file sharing)

2 participants