fix: Add missing /lms prefix to assignment submission notification links
Fixes #1969 - Added /lms prefix to notification link in lms_assignment_submission.py - Added /lms prefix to assessment URL in utils.py - Ensures consistent routing with other notification links (courses, billing, etc.) - Resolves 404 errors when users click 'View' on assignment submission notifications
This commit is contained in:
@@ -73,13 +73,7 @@ class LMSAssignmentSubmission(Document):
|
||||
"for_user": self.owner,
|
||||
"from_user": self.evaluator,
|
||||
"type": "Alert",
|
||||
"link": f"/assignment-submission/{self.assignment}/{self.name}",
|
||||
}
|
||||
)
|
||||
make_notification_logs(notification, [self.member])
|
||||
|
||||
|
||||
@frappe.whitelist()
|
||||
"link": f"/lms/assignment-submission/{self.assignment}/{self.name}",
|
||||
def upload_assignment(
|
||||
assignment_attachment=None,
|
||||
answer=None,
|
||||
|
||||
@@ -1269,7 +1269,7 @@ def get_assignment_details(assessment, member):
|
||||
|
||||
assessment.edit_url = f"/assignments/{assessment.assessment_name}"
|
||||
submission_name = existing_submission if existing_submission else "new-submission"
|
||||
assessment.url = f"/assignment-submission/{assessment.assessment_name}/{submission_name}"
|
||||
assessment.url = f"/lms/assignment-submission/{assessment.assessment_name}/{submission_name}"
|
||||
|
||||
return assessment
|
||||
|
||||
|
||||
Reference in New Issue
Block a user