Add "QA Rating" filter to archived item list #3136
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
|
@SocketSecurity ignore npm/replaywebpage@2.2.4 |
Change error detail to snake case for front-end localization Co-authored-by: Tessa Walsh <tessa@bitarchivist.net>
Co-authored-by: Tessa Walsh <tessa@bitarchivist.net>
deleted in cleanup
…github.com/webrecorder/browsertrix into 2875-task-add-filtering-by-qa-review-score
idk why else they would still be showing up in search value results
SuaYoo
left a comment
SuaYoo
left a comment
There was a problem hiding this comment.
Very cool idea! Left some small suggestions.
In general, I think it makes more sense to name this filter "QA Rating"/"Filter by QA Rating". The filter being named "QA Review" makes me think I should be able to filter by reviewed/not reviewed. "Rating" also makes more sense with the range slider. We do use this terminology and label the radio buttons as "Rate this crawl":
Fixes to missed tailwind classes, types, and improved url state parsing Co-authored-by: sua yoo <sua@webrecorder.org>
emma-sg
changed the title
|
Updated filter name to "QA Rating" 
|
tw4l
left a comment
tw4l
left a comment
There was a problem hiding this comment.
Nice work! Very nice addition to the frontend, and thanks for the crawlconfig search values fix :)
Co-authored-by: Tessa Walsh <tessa@bitarchivist.net>
Co-authored-by: Tessa Walsh <tessa@bitarchivist.net>
Closes #2875
Changes
Adds a filter for review state to the archived item list.
The filter uses a Web Awesome component, so this PR includes importing the Web Awesome CSS as well as its Slider component, which adds about 216kb of code to the final output.
Also excludes inactive workflows from the search values query, which will prevent searching for deleted workflows.
Demo
Screen.Recording.2026-01-26.at.1.26.05.PM.mov
Testing
Tested locally, and back-end changes are tested via integration test