Skip to content

Revert "Restore Harvard Van Gogh image to primary fixture"#4221

Merged
enriquediaz merged 1 commit intomainfrom
revert-4173-add-back-harvard
Jan 29, 2026
Merged

Revert "Restore Harvard Van Gogh image to primary fixture"#4221
enriquediaz merged 1 commit intomainfrom
revert-4173-add-back-harvard

Conversation

@marlo-longley
Copy link
Member

@marlo-longley marlo-longley commented Jan 20, 2026

Reverts #4173
This is broken again. It is keeping all other PRs from passing tests.
We should probably stop using this manifest, but the Van Gogh images are iconic to Mirador so we keep coming back...

@codecov
Copy link

codecov bot commented Jan 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.3%. Comparing base (ad08947) to head (1e068fe).
⚠️ Report is 21 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #4221     +/-   ##
=======================================
+ Coverage   95.1%   95.3%   +0.2%     
=======================================
  Files        325     326      +1     
  Lines      16177   16200     +23     
  Branches    2546    2559     +13     
=======================================
+ Hits       15376   15423     +47     
+ Misses       796     772     -24     
  Partials       5       5             

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@marlo-longley marlo-longley marked this pull request as ready for review January 20, 2026 18:22
@marlo-longley marlo-longley moved this to Needs PR Review in Accessibility + UX Jan 20, 2026
@enriquediaz
Copy link
Collaborator

Harvard reporting in. It's likely legit requests for manifests are getting caught in the net along with bot/non-human traffic once again. The story develops almost daily. I'm inquiring with our network operations team to learn more about what can be done here.

@marlo-longley
Copy link
Member Author

@enriquediaz thanks! If after a while it's seeming difficult on your end, we can always revert and swap back.

@enriquediaz enriquediaz self-requested a review January 29, 2026 17:04
Copy link
Collaborator

@enriquediaz enriquediaz left a comment

Choose a reason for hiding this comment

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

Approving and merging.

@enriquediaz enriquediaz merged commit ff5ec95 into main Jan 29, 2026
11 of 12 checks passed
@github-project-automation github-project-automation bot moved this from Needs PR Review to Done in Accessibility + UX Jan 29, 2026
@enriquediaz enriquediaz deleted the revert-4173-add-back-harvard branch January 29, 2026 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants