guests icon indicating copy to clipboard operation
guests copied to clipboard

Guests created via Talk don't receive a welcome mail

Open SystemKeeper opened this issue 1 year ago • 6 comments

Steps to reproduce:

  • Create a talk conversation
  • Enter a mail address in the share tab and select "Invite guest"
  • In the modal add name and create the guest

Expected:

  • The guest receives a mail that he was invited to a talk conversation and has the ability to reset the password

Current:

  • No mail is send to the guest, so the person who created the guest needs to manually reach out to them and tell them to use the "Reset password" functionality to create the credentials

Additional: Creating guests via file share is working correctly, the guest receive a mail. This is done inside the Hook.php: https://github.com/nextcloud/guests/blob/master/lib/Hooks.php

SystemKeeper avatar Jun 30 '23 05:06 SystemKeeper

Talk is using the PUT ocs/v2.php/…/api/v1/users endpoint, but yeah the GuestManager doesn't seem to send an email there. It only generates the password reset token: https://github.com/nextcloud/guests/blob/0ab2d2bb318b5eb40a19c0b763dd4ba3b87a53cd/lib/GuestManager.php#L104-L155

So also guests created via OCC don't receive their email atm.

nickvergessen avatar Jun 30 '23 08:06 nickvergessen

So I guess we need to move the mail logic to the GuestManager instead of reacting to a share hook?

SystemKeeper avatar Jun 30 '23 08:06 SystemKeeper

not move (it's in a MailService), but just invoke that one also from the GuestManager, yes, and then skip in the email parts that refer to a share, I guess

nickvergessen avatar Jun 30 '23 08:06 nickvergessen

Yes, that what I meant. Poor wording, sorry.

SystemKeeper avatar Jun 30 '23 09:06 SystemKeeper

Hello,

Still have this issue here. Is there any fix in backlog for this?

Th3l5D avatar Feb 21 '24 09:02 Th3l5D

Hey again,

I'm affraid this issue will fall in a black hole :D

Do you think it will be taken in account at some point?

Th3l5D avatar Feb 28 '24 16:02 Th3l5D