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.

Sign up for GitHub

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

Jump to bottom

[Community] Receiving mention notifications (@everyone) from spectated communities #14798

Closed
alexjba opened this issue May 15, 2024 · 2 comments · Fixed by status-im/status-go#5180
Closed

[Community] Receiving mention notifications (@everyone) from spectated communities #14798

alexjba opened this issue May 15, 2024 · 2 comments · Fixed by status-im/status-go#5180
Assignees
@friofry
Labels
backend-team
Milestone
2.30.0 Beta

Comments

@alexjba
Copy link
Contributor

alexjba commented May 15, 2024

Description

When spectating a community the @everyone mention will trigger a mention notification

Screenshot 2024-05-15 at 23 36 18

Steps to reproduce:

  1. Import an open community with request to join active
  2. Add a @everyone mention in the community from a member account

Expected behaviour:
The @everyone mention doesn't trigger a mention notification if the user is not a member

Actual behaviour:
The @everyone mention triggers a mention notification
@jrainville
Copy link
Member

Expected behaviour:
The @everyone mention doesn't trigger a mention notification if the user is not a member

I just want to validate the expectations before having someone work on this. It's a pretty simple fix, but I want to be sure it's what we want. I don't mid either way. cc @benjthayer

@benjthayer
Copy link

Expected behaviour:
The @everyone mention doesn't trigger a mention notification if the user is not a member

I just want to validate the expectations before having someone work on this. It's a pretty simple fix, but I want to be sure it's what we want. I don't mid either way. cc @benjthayer

Yes, that's correct, if you're not a member you should not receive notifications

@jrainville jrainville added this to the 2.30.0 Beta milestone May 16, 2024
@friofry friofry mentioned this issue May 17, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@friofry friofry

Labels
backend-team
Projects
Status Desktop/Mobile Board
Status: Done
Milestone
2.30.0 Beta
4 participants
@friofry @jrainville @alexjba @benjthayer