Skip to content

view only your own file level recordings#372

Merged
BryonLewis merged 1 commit intomainfrom
limit-file-annotation-viewer
Feb 23, 2026
Merged

view only your own file level recordings#372
BryonLewis merged 1 commit intomainfrom
limit-file-annotation-viewer

Conversation

@BryonLewis
Copy link
Collaborator

Makes it so user's only see their own file annotations in the viewer instead of all user's.

@BryonLewis BryonLewis requested a review from naglepuff February 23, 2026 18:19
Comment on lines +663 to +665
fileAnnotations = RecordingAnnotation.objects.filter(
recording=recording_id, owner=request.user
).order_by('confidence')
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this logic be true for superusers and recording owners, or should they be able to see all annotations?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can worry about that in a subsequent PR. I didn't want to clutter this small one.
If you remember my pulse annotations had a mode if you were the owner of admin you could see all other annotations. We can do something like that at some point for this.

@BryonLewis BryonLewis merged commit 3fa3ad9 into main Feb 23, 2026
6 checks passed
@BryonLewis BryonLewis deleted the limit-file-annotation-viewer branch February 23, 2026 18:33
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.

2 participants