Skip to content

IBX-12339: Email sending not working : Invite user - #257

Open
vidarl wants to merge 1 commit into
masterfrom
IBX-12339_Email_sending_not_working_Invite_user
Open

IBX-12339: Email sending not working : Invite user#257
vidarl wants to merge 1 commit into
masterfrom
IBX-12339_Email_sending_not_working_Invite_user

Conversation

@vidarl

@vidarl vidarl commented Aug 27, 2026

Copy link
Copy Markdown
Contributor
🎫 Issue IBX-12339

Related PRs:

Strictly speaking not related, but needed in order to make the invite user functionality work end-to-end (IBX-12340):

Description:

Ibexa\AdminUi\Notifier\UserInvitation decorates Ibexa\Contracts\User\Invitation\InvitationSender
and only send emails when notifications.subscriptions contains
Ibexa\AdminUi\Notifier\Notification\UserInvitation. No recipe subscribes that class: the shipped
config/packages/ibexa.yaml subscribes Ibexa\Contracts\User\Notification\UserInvitation, which the
decorator never reads. ( The difference is Ibexa\AdminUi... vs Ibexa\Contracts.... in the namespace)

So on a clean install, the invite-user functionality never sends the invitation e-mail.

This issue is similar to IBX-11241 (UserPasswordReset email never sent) and was fixed the same way in #236.

Also adding FormSubmitted on editions where that is applicable. The subscriptions array
in default and admin_group won't be merged. Thus, it needs to be copied to admin_group.
Submitting forms from the back office might not be common, but it can happen.

Applied to 5.0 and 6.0 of all four editions. FormSubmitted only goes to experience and commerce,
since ibexa/form-builder is not part of oss or headless.

For QA:

On a clean 5.0.10 (or 6.0) install with a working MAILER_DSN:

  1. Before the change: invite someone from Admin → Users → Guest accounts → Invite members. The
    invitation row appears in ibexa_user_invitation with site_access_name = admin, and no mail is
    sent.
  2. After the change (composer recipes:install ibexa/<edition> --force, then cache:clear): repeat
    the invitation. The mail arrives, containing a link to
    /admin/from-invite/register/{hash}.

Documentation:

@github-actions

Copy link
Copy Markdown

Thanks for contribution! 🎉

To test the changes please execute:

composer config extra.symfony.endpoint https://api.github.com/repos/ibexa/recipes-dev/contents/index.json?ref=flex/pull-257

before executing the recipes.

@sonarqubecloud

Copy link
Copy Markdown

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.

3 participants