Remove django-settings-context-processor and define better ways to handl the values it passes #434

Closed
opened 2021-05-15 15:42:50 +02:00 by nik · 3 comments
Owner

Too trivial to have a dependency for it. Also, seems like an anti-pattern. The ADMINS list is used in error templates and could be injected from the error views; the DEBUG is only used on the status page and could be injected there.

If wer want to make things like ocnfigured admins availlable as a template API for apps, we should provide selective templatetags for it.

Too trivial to have a dependency for it. Also, seems like an anti-pattern. The ADMINS list is used in error templates and could be injected from the error views; the DEBUG is only used on the status page and could be injected there. If wer want to make things like ocnfigured admins availlable as a template API for apps, we should provide selective templatetags for it.
Owner

created merge request !2245 to address this issue

created merge request !2245 to address this issue
Owner

mentioned in merge request !2245

mentioned in merge request !2245
nik closed this issue 2021-05-15 18:51:43 +02:00
Author
Owner

mentioned in commit 780a205ef5

mentioned in commit 780a205ef503a504bb01d1c27147bc7815ee4f22
Sign in to join this conversation.
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
aleksis/AlekSIS-Core#434
No description provided.