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

noteAllDescendants also disables push notifications for direct replies #1844

Open
1 task done
ekzyis opened this issue Jan 25, 2025 · 0 comments
Open
1 task done

noteAllDescendants also disables push notifications for direct replies #1844

ekzyis opened this issue Jan 25, 2025 · 0 comments
Labels

Comments

@ekzyis
Copy link
Member

ekzyis commented Jan 25, 2025

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

We currently only send out push notifications for replies if noteAllDescendants is set:

const createUserFilter = (tag) => {
// filter users by notification settings
const tagMap = {
REPLY: 'noteAllDescendants',
MENTION: 'noteMentions',
ITEM_MENTION: 'noteItemMentions',
TIP: 'noteItemSats',
FORWARDEDTIP: 'noteForwardedSats',
REFERRAL: 'noteInvites',
INVITE: 'noteInvites',
EARN: 'noteEarning',
DEPOSIT: 'noteDeposits',
WITHDRAWAL: 'noteWithdrawals',
STREAK: 'noteCowboyHat'
}
const key = tagMap[tag.split('-')[0]]
return key ? { user: { [key]: true } } : undefined
}

However, direct replies always show up in /notifications so push notifications are inconsistent with the notifications we show there.

To fix this, we would need to always send a push notification for every reply.

Screenshots

No response

Steps To Reproduce

  1. Enable push notifications
  2. Disable someone replies to someone who replied to me in settings
  3. Reply to yourself with anon
  4. No push notification received but notification in /notifications

Expected behavior

Push notification received

Logs

No response

Device information

No response

Additional context

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant