Skip to content

SCIX-817 feat(search-facet): filter collections facet to exclude astrophysics, planetary, and heliophysics#777

Merged
thostetler merged 2 commits intoadsabs:masterfrom
thostetler:scix-817-filter-collections-facet
Jan 24, 2026
Merged

SCIX-817 feat(search-facet): filter collections facet to exclude astrophysics, planetary, and heliophysics#777
thostetler merged 2 commits intoadsabs:masterfrom
thostetler:scix-817-filter-collections-facet

Conversation

@thostetler
Copy link
Member

@thostetler thostetler commented Jan 22, 2026

Filter out planetary, helio, astrophysics from the collections facet

  • Add filter to collections facet config to show only astronomy, physics, general, earthscience
  • Update mock data to exercise the filter in tests
image

@codecov
Copy link

codecov bot commented Jan 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.0%. Comparing base (4b6a58c) to head (e87b1d0).

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #777     +/-   ##
========================================
+ Coverage    61.0%   61.0%   +0.1%     
========================================
  Files         301     301             
  Lines       35169   35176      +7     
  Branches     1515    1515             
========================================
+ Hits        21438   21445      +7     
  Misses      13697   13697             
  Partials       34      34             
Files with missing lines Coverage Δ
src/api/search/types.ts 100.0% <100.0%> (ø)
src/components/SearchFacet/config.ts 100.0% <100.0%> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@thostetler thostetler changed the title SCIX-817 feat(search-facet): filter collections facet to exclude legacy databases SCIX-817 feat(search-facet): filter collections facet to exclude astrophysics, planetary, and heliophysics Jan 22, 2026
…cy databases

The API returns 7 database values but we only want to display 4 in the
Collections facet UI.

- Add filter property to collections config to include only astronomy, physics, general, earthscience
- Expand Database type to include all API values (astrophysics, heliophysics, planetary)
- Update mock data to reflect real API response with all 7 values
@thostetler thostetler force-pushed the scix-817-filter-collections-facet branch from fc73f4a to e87b1d0 Compare January 22, 2026 21:41
@thostetler thostetler requested a review from shinyichen January 22, 2026 21:44
@thostetler thostetler marked this pull request as ready for review January 22, 2026 21:44
@thostetler
Copy link
Member Author

@kelockhart please check the photo in the main description, is this the expected behavior?

@kelockhart
Copy link
Member

@kelockhart please check the photo in the main description, is this the expected behavior?

Yep!

@thostetler thostetler merged commit 0488b04 into adsabs:master Jan 24, 2026
5 checks passed
@thostetler thostetler deleted the scix-817-filter-collections-facet branch January 24, 2026 02:44
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.

3 participants