Skip to content

refactor: replace comment read receipts with per-task cursors - #659

Open
b0ink wants to merge 13 commits into
doubtfire-lms:11.0.xfrom
b0ink:refactor/read-receipt-cursor
Open

refactor: replace comment read receipts with per-task cursors#659
b0ink wants to merge 13 commits into
doubtfire-lms:11.0.xfrom
b0ink:refactor/read-receipt-cursor

Conversation

@b0ink

@b0ink b0ink commented Jul 28, 2026

Copy link
Copy Markdown
Member

Actual unit 12 measurements:

Metric Old receipts New cursors
Rows 25,751,861 2,285,027
Rows per task 823 73
Reduction 91.1%
Improvement 11.3× fewer rows
Table Data + indexes
Old receipts 5.29 GB
New cursors 632 MB
Reduction 88.1%

TODO: Automated comments break this cursor design - OnTrack should have a "SYSTEM" user account that all automated comments send from. Eg. overseer assessment reports, task status updates (only automated task status updates?), project plan comments? anything that currently automatically gets marked as read by the receipient should be from the SYSTEM

Task comments now have a "audience" field, used to identify who the comment is intended for. "Students" are the audience for automated comments, "noone" for status updates, and either "staff" or "students" for text comments depending on the recipient. This allows the comment cursor to display up to a student's text comment, and the inbox will ignore any status updates, project plan changes, overseer assessments that would otherwise normally be marked as read by all parties

@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant