Skip to content

Include institution sources in the full text sources dropdown - #928

Merged
shinyichen merged 1 commit into
adsabs:masterfrom
shinyichen:SCIX-905-search-results-missing-institution-sources
Sep 3, 2026
Merged

Include institution sources in the full text sources dropdown#928
shinyichen merged 1 commit into
adsabs:masterfrom
shinyichen:SCIX-905-search-results-missing-institution-sources

Conversation

@shinyichen

Copy link
Copy Markdown
Member

Include institution sources in the full text sources dropdown in search results

Screenshot 2026-09-03 at 10 21 47 AM

@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.9%. Comparing base (4c73059) to head (07e62d4).

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #928     +/-   ##
========================================
+ Coverage    67.8%   67.9%   +0.1%     
========================================
  Files         378     378             
  Lines       44164   44165      +1     
  Branches     2377    2379      +2     
========================================
+ Hits        29937   29948     +11     
+ Misses      14179   14169     -10     
  Partials       48      48             
Files with missing lines Coverage Δ
src/api/search/models.ts 53.8% <100.0%> (+0.2%) ⬆️
...mponents/ResultList/Item/ItemResourceDropdowns.tsx 77.8% <100.0%> (ø)

... and 1 file 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.

@thostetler thostetler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@shinyichen
shinyichen merged commit 936dcfb into adsabs:master Sep 3, 2026
5 checks passed
@shinyichen
shinyichen deleted the SCIX-905-search-results-missing-institution-sources branch September 3, 2026 18:07
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