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

allow apps to not include the user in a notification #698

Merged
merged 1 commit into from
Jan 16, 2024
Merged

allow apps to not include the user in a notification #698

merged 1 commit into from
Jan 16, 2024

Commits on Jan 15, 2024

  1. allow apps to not include the user in a notification 

    While some may find the current OS user helpful, in other contexts, that user is randomly assigned and not material to the notification itself. Consider a deployment enviroment like Heroku, which will use a different OS user for each deployable, and each deploy.

It may also cause grouping issues in services like Sentry, which use the exception / notification content to determine whether the situation has occurred already.

This does not change the default behavior, but rather introduces a new option to skip including that information in notifications, for those who want to.
    @agrobbin
    agrobbin committed Jan 15, 2024
    Configuration menu
    Copy the full SHA
    feb8ed1 View commit details
    Browse the repository at this point in the history