Other — User submission
complete
P
Parent 019e1543-395
Category:
OtherType:
User submissionUser role:
parentUser feedback:
When any thing are suggested, like topics or courses with bullet-points, it's better to always show the options numbers, like 1 to 9 or more and ask them to input the number as the choice than text
P
Patrick Jean
updated the status to
complete
Verified non-reproducible at commit 43de2d6c; closing the 404 document link issue as fixed. The frontend correctly prepends persona basePath (/student/ or /parent/) to canvas_url in all code paths (MessageActionBar.tsx:272, DocumentSavedCard.tsx:38, DocumentsPageContent.tsx:340). PR #2620 closed — fix was already shipped in a prior change. (automated investigator run 28552264973)
S
Sage Investigator
Re-verification (2026-07-01, commit 43de2d6c, automated investigator run 28546438618):
Document link 404 bug still present — save_document.py
lines 102 and 185 generate canvas_url=f"/documents/{doc.document_id}"
without persona path prefix. Open fix PR: https://github.com/momentiq-ai/sage3c/pull/2620S
Sage Investigator
Re-verification (2026-07-01, commit 43de2d6c, manual session):
Document link 404 bug still present — save_document.py
lines 102 and 185 generate canvas_url=f"/documents/{doc.document_id}"
without persona path prefix (e.g. /student/documents/...
or /parent/documents/...
). Active fix PR: https://github.com/momentiq-ai/sage3c/pull/2620 (open, re-verified).S
Sage Investigator
Re-verification (2026-07-01, commit 43de2d6c, manual session):
Document link 404 bug still present — save_document.py
lines 102 and 185 (now at backend/domains/documents/use_cases/save_document.py
) still generate canvas_url
without persona path prefix. Active fix PR: https://github.com/momentiq-ai/sage3c/pull/2620P
Patrick Jean
updated the status to
in progress
Updating status to "in progress" — active fix PR exists: https://github.com/momentiq-ai/sage3c/pull/2620. Bug re-verified still present at commit 43de2d6c (feedback-investigator run 28496300499).
S
Sage Investigator
Re-verification (2026-06-30, commit 43de2d6c, automated investigator run 28480216826):
Bug still present — save_document.py
lines 102 and 185 generate canvas_url=f"/documents/{doc.document_id}"
without a persona path prefix (e.g., /student/documents/...
), causing 404 when document links are clicked. Active fix PR: https://github.com/momentiq-ai/sage3c/pull/2620S
Sage Investigator
Re-verification (2026-06-30, commit 43de2d6c, automated investigator run 28474731643):
Bug still present — save_document.py
lines 102 and 185 generate canvas_url=f"/documents/{doc.document_id}"
without the persona prefix (/student/
or /parent/
), causing 404 when clicked. Open fix PR #2620 re-verified. Frontend routes require /student/documents/[id]
or /parent/documents/[id]
.P
Patrick Jean
Re-verification (2026-06-30, commit 43de2d6c, automated investigator run 28468136702):
Bug still present — save_document.py
lines 102 and 185 generate canvas_url=f"/documents/{doc.document_id}"
without persona path prefix (/student/
or /parent/
). Frontend routes at app/(protected)/student/documents/[documentId]/page.tsx
and app/(protected)/parent/documents/[documentId]/page.tsx
require the prefix. Open PR #2620 addresses this fix but has not been merged yet. Re-verified comment added to PR.S
Sage Investigator
Re-verification (2026-06-29, commit e7f89c97, automated investigator run 28395215780):
Bug still present — save_document.py
lines 102 and 185 generate canvas_url=f"/documents/{doc.document_id}"
without persona path prefix, causing 404 when students/parents click document links. Open fix PR: https://github.com/momentiq-ai/sage3c/pull/2620S
Sage Investigator
Re-verification (2026-06-29, commit e7f89c97, automated investigator run 28389526736):
Bug still present — save_document.py
lines 102 and 185 generate canvas_url=f"/documents/{doc.document_id}"
without persona path prefix. Additionally confirmed documents/api/routes.py
line 572 has the same pattern. No frontend route exists at /documents/{id}
(only /student/documents/
and /parent/documents/
). Active fix PR: https://github.com/momentiq-ai/sage3c/pull/2620Load More
→