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

Allow management of Django's ADMINS setting #65

Merged
merged 1 commit into from
Oct 2, 2023
Merged

Conversation

smortex
Copy link
Member

@smortex smortex commented Oct 2, 2023

ADMINS is a Django setting for people who should get error notifications:

A list of all the people who get code error notifications. When
DEBUG=False and AdminEmailHandler is configured in LOGGING (done by
default), Django emails these people the details of exceptions raised
in the request/response cycle.

The default value from Taiga is the invalid address example@example.com. Add parameters to use custom address and default to an empty list of administrators.

@smortex smortex added the enhancement New feature or request label Oct 2, 2023
ADMINS is a Django setting for people who should get error
notifications:

> A list of all the people who get code error notifications. When
> DEBUG=False and AdminEmailHandler is configured in LOGGING (done by
> default), Django emails these people the details of exceptions raised
> in the request/response cycle.

The default value from Taiga is the invalid address
example@example.com.  Add parameters to use custom address and default
to an empty list of administrators.
@smortex smortex marked this pull request as ready for review October 2, 2023 03:36
@smortex smortex requested a review from a team as a code owner October 2, 2023 03:36
@smortex smortex changed the title Allow management of ADMINS Allow management of Django's ADMINS setting Oct 2, 2023
@neomilium neomilium merged commit adeee16 into master Oct 2, 2023
6 checks passed
@neomilium neomilium deleted the manage-admins branch October 2, 2023 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants