Skip to content

Bulk remove items from library from search page - #916

Open
shinyichen wants to merge 1 commit into
adsabs:masterfrom
shinyichen:SCIX-903-bulk-delete-from-libraries
Open

Bulk remove items from library from search page#916
shinyichen wants to merge 1 commit into
adsabs:masterfrom
shinyichen:SCIX-903-bulk-delete-from-libraries

Conversation

@shinyichen

Copy link
Copy Markdown
Member

Add "remove from library" in the bulk action in search results page.
Refactor add to library modal to handle remove from library.

Screenshot 2026-08-12 at 9 44 42 AM

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 74.41860% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.1%. Comparing base (488478e) to head (31a8148).

Files with missing lines Patch % Lines
...mponents/Libraries/AddToRemoveFromLibraryModal.tsx 72.5% 11 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           master    #916     +/-   ##
========================================
- Coverage    68.1%   68.1%   -0.0%     
========================================
  Files         357     357             
  Lines       42007   42019     +12     
  Branches     2218    2219      +1     
========================================
- Hits        28593   28589      -4     
- Misses      13365   13381     +16     
  Partials       49      49             
Files with missing lines Coverage Δ
src/components/ResultList/ListActions.tsx 78.6% <100.0%> (+0.1%) ⬆️
...mponents/Libraries/AddToRemoveFromLibraryModal.tsx 83.4% <72.5%> (ø)

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@shinyichen
shinyichen requested a review from thostetler August 13, 2026 16:18
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