search

devilry / devilry-django

Devilry project main repository
http://devilry.org
BSD 3-Clause "New" or "Revised" License
51 stars 24 forks source link

Incorrect role path in notification template #1216

Closed torgeirl closed 2 years ago

torgeirl commented 2 years ago

It seems the notification templating incorrectly adds the /examiner/ path to devilry_email_comment_detail_url when notifying administrators .

Example: when another administrator comments an assignment the «An admin added a new comment for {assignment name}» notification's devilry_email_comment_detail_url will use an /examiner/ path. Unless the admin receiving the notification also is the student group's examiner the URL will give HTTP 403 permission denied.

Workaround The admin can manually replace /examiner/ with /admin/.

StianJul commented 2 years ago

@torgeirl, this is now fixed and will be available with the 5.4-release.