Skip to content

Only notify user who started BulkImport

James Nutt requested to merge jnutt/only-email-import-triggerer into master

What does this MR do and why?

Only notify user who started BulkImport

Only the user who started the bulk import can view the bulk import history, so it only makes sense to notify them on import completion.

This is to reduce confusion when a group owner is emailed a link to something they cannot access.

Changelog: changed

Related issue: #487324 (closed)

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to set up and validate locally

  1. Enable Direct Transfer and local requests via your Rails console:
ApplicationSetting.first.update(
  allow_local_requests_from_web_hooks_and_services: true,
  bulk_import_enabled: true
)
  1. Import an existing group from your instance into another namespace.
  2. When the import completes, check for the e-mail in letter opener.
Edited by James Nutt

Merge request reports