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

Backup notifications need to @ the user intended #64

Open
tescher opened this issue Aug 20, 2022 · 1 comment
Open

Backup notifications need to @ the user intended #64

tescher opened this issue Aug 20, 2022 · 1 comment

Comments

@tescher
Copy link
Collaborator

tescher commented Aug 20, 2022

It looks like if someone submits a bounty for review, and the creator doesn't have DMs turned on, it sends the message to the bounty channel. This is fine, but it also needs to @ the creator so they are notified. Also check all other notifications to make sure they do this.
@tescher
Copy link
Collaborator Author

tescher commented Apr 19, 2023 •
edited
Loading

NotificationPermissionError needs to include the intended user in the content for the error so it appears in the backup message, and probably a link to the bounty

@tescher tescher mentioned this issue Apr 23, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tescher