Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an option for whether to receive a direct message when invited users join #20398

Closed
xeruf opened this issue Nov 30, 2021 · 26 comments · Fixed by #28819
Closed

Add an option for whether to receive a direct message when invited users join #20398

xeruf opened this issue Nov 30, 2021 · 26 comments · Fixed by #28819

Comments

@xeruf
Copy link

xeruf commented Nov 30, 2021

This issue description has been updated by @alya to include a specific design and by @laurynmm for the change from "private message" to "direct message" in the UI.

At present, when a user's invitation to Zulip is accepted, they receive a direct message announcing this. However, some users may not want to receive such a message, e.g. because they are already following new user notifications in a stream, or they simply don't need to know when new users join.

To address this, we should add a feature to turn off direct message notifications when sending an invitation. In particular, the user invite modal should have the following checkbox below the Emails text box:

[ ] Send me a direct message when my invitation is accepted.

Notes:

  • The checkbox should default to the previous state, which in turn defaults to enabled.
  • The checkbox should only be shown when inviting by email, not when creating an invite link. (We don't send notifications when invite links are used.)
  • If a user flips back and forth between inviting by email and generating a link, the current state of the checkbox should be preserved.

Original issue:

Allow Muting Notification Bot
I am annoyed that I get double-notified when a new user joins:
Through the channel topic for new users and a PM from the notification bot, as I am personally responsible for invitations.
I checked out Mute a User and https://docs.zulip.com/help/configure-notification-bot, but neither applies to PMs from the Notification Bot...

@timabbott
Copy link
Sponsor Member

timabbott commented Dec 1, 2021

@xeruf thanks for the report! I think don't think muting the notification bot is the right solution for this problem since it might disable other notifications you want; instead I think a personal notification setting for whether you get a PM when a new user joins who you invited makes more sense.

@alya thoughts on that design question?

@zulipbot
Copy link
Member

zulipbot commented Dec 5, 2021

Hello @JoshuaPius!

Thanks for your interest in Zulip! You have attempted to claim an issue without the labels "help wanted", "good first issue". Since you're a new contributor, you can only claim and submit pull requests for issues with the help wanted or good first issue labels.

If this is your first time here, we recommend reading our guide for new contributors before getting started.

@yashd26
Copy link
Collaborator

yashd26 commented Jan 7, 2022

@zulipbot claim

@zulipbot
Copy link
Member

zulipbot commented Jan 7, 2022

Hello @algo-byte, it looks like you've currently claimed 1 issue in this repository. We encourage new contributors to focus their efforts on at most 1 issue at a time, so please complete your work on your other claimed issues before trying to claim this issue again.

We look forward to your valuable contributions!

@yashd26
Copy link
Collaborator

yashd26 commented Jan 20, 2022

@alya can you please tell me if this issue is open to work on?

@alya
Copy link
Contributor

alya commented Jan 21, 2022

@algo-byte This issue wasn't really ready, as we didn't have a worked out design.

In particular, I think the right approach is to ask the user when they send the invitation, rather than making it a display setting.

@alya
Copy link
Contributor

alya commented Jan 21, 2022

I can write up a description soon and mark it as "help wanted" at that point.

@yashd26
Copy link
Collaborator

yashd26 commented Jan 21, 2022

Thank you for confirming, will make sure to check out the labels first before working on any issue next time.

@yashd26
Copy link
Collaborator

yashd26 commented Jan 21, 2022

@sahil839 thank you so much for helping me on this issue. Learned a lot from you and from this issue.

@alya alya changed the title Allow Muting Notification Bot Add an option for whether to receive a PM when invited users join Jan 21, 2022
@alya
Copy link
Contributor

alya commented Jan 21, 2022

I've updated the issue, so it's ready to be worked on now.

@yashd26
Copy link
Collaborator

yashd26 commented Jan 22, 2022

@alya does that mean that for every invitee, the referrer can decide if he/she wants a PM or not?

@xeruf
Copy link
Author

xeruf commented Jan 23, 2022

Little design annotation:

The checkbox should default to the previous state, which in turn defaults to enabled on by default.

@yashd26
Copy link
Collaborator

yashd26 commented Jan 25, 2022

@zulipbot claim

@alya
Copy link
Contributor

alya commented Jan 25, 2022

@algo-byte Not for every invitee, but every time the user is sending out a bundle of invites. If you want to get a message when A joins but not B, you'd have to invite them separately.

@alya
Copy link
Contributor

alya commented Jan 25, 2022

Little design annotation:

The checkbox should default to the previous state, which in turn defaults to enabled on by default.

Good idea, thanks! Updated accordingly.

If the implementation for this is complicated, we can discuss whether it's worth maintaining the state information. CC @timabbott

@yashd26
Copy link
Collaborator

yashd26 commented Jan 27, 2022

@alya does this look good?

Capture

@yashd26
Copy link
Collaborator

yashd26 commented Jan 27, 2022

@alya @timabbott I am having a little difficulty in preserving the default state of the checkbox. I tried finding the event handler that opens and closes user invite modal but couldn't find it. Also tried finding the #/invite url that gets called when we open the user invite modal but I am unable to find the right files. Any help is greatly appreciated.

@alya
Copy link
Contributor

alya commented Jan 28, 2022

The screenshot looks good to me!

For the question above, you should post it in the #backend stream on chat.zulip.org so that more people have a chance to see it an help you out.

yashd26 added a commit to yashd26/zulip that referenced this issue Jan 29, 2022
Everytime an invitee joins any realm the referrer is informed about
invitee joining the realm through private message.

This commit adds a checkbox at invite modal which allows the user to
choose whether or not to get that private message. For that a new
`receive_private_message_on_invitee_signup` field is added to user_
notification_setting.

Fixes: zulip#20398
yashd26 added a commit to yashd26/zulip that referenced this issue Jan 29, 2022
Everytime an invitee joins any realm the referrer is informed about
invitee joining the realm through private message.

This commit adds a checkbox at invite modal which allows the user to
choose whether or not to get that private message. For that a new
`receive_private_message_on_invitee_signup` field is added to user_
notification_setting.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Mar 22, 2024
Previously, when a user's invitations were accepted,
they got notification from welcome bot.
However some user may not want this everytime.

Presently, there is a personal notification setting
which lets you decide if you want notification.

Fixes: zulip#20398.
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Mar 23, 2024
Previously, when a user's invitations were accepted,
they got notification from welcome bot.
However some user may not want this everytime.

Presently, there is a personal notification setting
which lets you decide if you want notification.

Fixes: zulip#20398.
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Mar 25, 2024
Previously, when a user's invitations were accepted,
they got notification from welcome bot.
However some user may not want this everytime.

Presently, there is a personal notification setting
which lets you decide if you want notification.

Fixes: zulip#20398.
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Mar 29, 2024
Previously, when a user's invitations were accepted,
they got notification from welcome bot.
However some user may not want this everytime.

Presently, there is a personal notification setting
which lets you decide if you want notification.

Fixes: zulip#20398.
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Mar 30, 2024
Previously, when a user's invitations were accepted,
they got notification from welcome bot.
However some user may not want this everytime.

Presently, there is a personal notification setting
which lets you decide if you want notification.

Fixes: zulip#20398.
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 1, 2024
Previously, when a user's invitations were accepted,
they got notification from welcome bot.
However some user may not want this everytime.

Presently, there is a personal notification setting
which lets you decide if you want notification.

Fixes: zulip#20398.
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 2, 2024
Previously, when a user's invitations were accepted,
they got notification from welcome bot.
However some user may not want this everytime.

Presently, there is a personal notification setting
which lets you decide if you want notification.

Fixes: zulip#20398.
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 3, 2024
…ations.

Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 3, 2024
…bot.

Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 3, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 3, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 5, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 11, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 11, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 13, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 18, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 23, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 23, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Apr 25, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue May 15, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue May 15, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue May 22, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue May 22, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Jun 28, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
timabbott pushed a commit to shashank-23002/zulip that referenced this issue Jul 2, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shashank-23002 added a commit to shashank-23002/zulip that referenced this issue Jul 4, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
shubhiscoding pushed a commit to shubhiscoding/zulip that referenced this issue Jul 15, 2024
Previously, when a referrer's invitation to Zulip was accepted,
they got a notification from notification-bot indicating
their invitation has been accepted.

This commit adds an option for referrer to decide
whether he wants to receive the direct notification
from the notification-bot.

Fixes: zulip#20398
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment