Skip to content

Conversation

@naglepuff
Copy link
Collaborator

@naglepuff naglepuff commented Jan 5, 2026

Vetting workflow PR

Fix #282

Changes

The following sections are the acceptance criteria according to the linked issue.

After labeling a file, the label appears in the list view for that reviewer.

Adds a new column to the Recordings tables (private and public) displaying a user's submitted label for each recording, if one exists.

Adds new UI elements to the recordings sidebar in the spectrogram view to display the same information.

A reviewer can click between files and easily compare their prior labels.

Users already have the ability to navigate between files from the spectrogram view.

@naglepuff naglepuff linked an issue Jan 5, 2026 that may be closed by this pull request
@naglepuff naglepuff marked this pull request as ready for review January 5, 2026 19:33
@naglepuff naglepuff requested a review from BryonLewis January 5, 2026 19:33
Copy link
Collaborator

@BryonLewis BryonLewis left a comment

Choose a reason for hiding this comment

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

Just one comment, The 'Annotation' Column in the Shared Recordings. Should that be disabled when the vetting and submission is turned on. I think right now a non-admin non-owner can view other annotations on a shared recording. I.E the mdi-info circle icon when hovering over it will show the annotations of other users.

@naglepuff
Copy link
Collaborator Author

Just one comment, The 'Annotation' Column in the Shared Recordings. Should that be disabled when the vetting and submission is turned on. I think right now a non-admin non-owner can view other annotations on a shared recording. I.E the mdi-info circle icon when hovering over it will show the annotations of other users.

Good point. I'll hide that in a follow-up PR where I'm hiding other columns as well.

@naglepuff naglepuff merged commit a253a0e into vetting-workflow Jan 6, 2026
4 checks passed
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.

[Vetting] Show reviewer own species label in recording list

2 participants