Skip to content

feat(forms): add submission notification emails with external recipients and PDF attachments#3251

Open
QuiteBitter wants to merge 3 commits intonextcloud:mainfrom
AdventistenFiD:upstream-ready/05-owner-notification-pdf
Open

feat(forms): add submission notification emails with external recipients and PDF attachments#3251
QuiteBitter wants to merge 3 commits intonextcloud:mainfrom
AdventistenFiD:upstream-ready/05-owner-notification-pdf

Conversation

@QuiteBitter
Copy link
Copy Markdown

What this changes

  • Adds submission notification mails for new responses using the configured system mail account.
  • Lets form owners opt into owner notifications and configure additional external notification recipients per form.
  • Lets forms attach a generated PDF copy of the submission to those notification mails.
  • Validates and normalizes configured recipient addresses in the API and settings UI, and covers the notification / attachment flow with unit and integration tests.

Related issues

Notes

  • Additional recipients are independent of the owner-notification switch, so a form can notify external addresses without mailing the owner.
  • The mail body keeps the inline summary focused on short and long text answers; enabling the PDF attachment adds a portable copy of the captured submission data for forwarding and archiving.
  • Leaves generated locale catalogs out of the PR because Nextcloud translations are handled through Transifex.

Validation

  • Nextcloud 33: unit and integration suites passed.
  • Nextcloud 31: unit and integration suites passed.

@QuiteBitter QuiteBitter force-pushed the upstream-ready/05-owner-notification-pdf branch from 38f5633 to 4c15876 Compare April 1, 2026 02:04
Signed-off-by: Robert <admin@ro-bytes.de>
Signed-off-by: Robert <admin@ro-bytes.de>
Signed-off-by: Robert <admin@ro-bytes.de>
@QuiteBitter QuiteBitter force-pushed the upstream-ready/05-owner-notification-pdf branch from 4c15876 to 847e043 Compare April 1, 2026 02:18
@Chartman123
Copy link
Copy Markdown
Collaborator

Thanks for your PR! Please understand that we're currently migrating to Vue3 and won't merge new features until this has finished to minimize extra work.

@QuiteBitter
Copy link
Copy Markdown
Author

Thanks for your PR! Please understand that we're currently migrating to Vue3 and won't merge new features until this has finished to minimize extra work.

Okey, I will wait for this to complete :) good luck! After completion, I'll check what's needed to be done on my side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants