Skip to content

fix(search): use arxiv shortcode in arxiv_class example#829

Merged
thostetler merged 1 commit intoadsabs:masterfrom
thostetler:fix/arxiv-class-shortcodes
Mar 25, 2026
Merged

fix(search): use arxiv shortcode in arxiv_class example#829
thostetler merged 1 commit intoadsabs:masterfrom
thostetler:fix/arxiv-class-shortcodes

Conversation

@thostetler
Copy link
Copy Markdown
Member

@thostetler thostetler commented Mar 23, 2026

  • Replace long-form arxiv class name in the arxiv_class search term example with the short code format (astro-ph.IM)

https://adsabs.slack.com/archives/C041TCL9F/p1773259903143839

@thostetler thostetler marked this pull request as ready for review March 23, 2026 16:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the SearchBar search-term help content to show arXiv categories using the canonical shortcode format, making the arxiv_class example align with typical query inputs.

Changes:

  • Replaced the long-form arXiv class name example with the short code (astro-ph.IM) for arxiv_class.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.4%. Comparing base (bb93ce0) to head (7be21b2).

Additional details and impacted files
@@          Coverage Diff           @@
##           master    #829   +/-   ##
======================================
  Coverage    62.4%   62.4%           
======================================
  Files         317     317           
  Lines       36565   36565           
  Branches     1670    1671    +1     
======================================
  Hits        22816   22816           
  Misses      13711   13711           
  Partials       38      38           
Files with missing lines Coverage Δ
src/components/SearchBar/models.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 merged commit ae22f5c into adsabs:master Mar 25, 2026
9 checks passed
@thostetler thostetler deleted the fix/arxiv-class-shortcodes branch March 25, 2026 14:00
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