Skip to content

docs: fix stale gallery link and broken DOI badge - #768

Merged
timtreis merged 2 commits into
mainfrom
fix/notebooks-repo-rename
Aug 13, 2026
Merged

docs: fix stale gallery link and broken DOI badge#768
timtreis merged 2 commits into
mainfrom
fix/notebooks-repo-rename

Conversation

@timtreis

@timtreis timtreis commented Aug 13, 2026

Copy link
Copy Markdown
Member

Two small docs fixes.

1. Stale gallery source link

The notebooks submodule was renamed scverse/spatialdata-plot-notebooks -> scverse/spatialdata-plot-tutorials (already reflected in .gitmodules), but docs/gallery.md still linked the old repo in its Sources line. GitHub redirects the old URL, so the link worked but was stale.

2. Broken DOI badge

The README DOI badge used the redirect-style form (zenodo.org/badge/<repoid>.svg), which returns a 302. GitHub's camo image proxy caches that redirect instead of following it to the SVG, so the badge renders as a broken image. Switched to the direct DOI badge form used by scverse/spatialdata (zenodo.org/badge/DOI/10.5281/zenodo.<id>.svg -> 200 image/svg+xml, no redirect).

… repo

The notebooks submodule was renamed scverse/spatialdata-plot-notebooks ->
scverse/spatialdata-plot-tutorials (see .gitmodules), but the gallery
Sources link still pointed at the old repo. GitHub redirects it, so it
was stale rather than broken.
The redirect-style badge (zenodo.org/badge/<repoid>.svg) returns a 302,
which GitHub's camo image proxy renders as a broken image. Switch to the
direct DOI badge form used by scverse/spatialdata (200 image/svg+xml, no
redirect).
@timtreis timtreis changed the title docs: fix stale gallery link after notebooks repo rename docs: fix stale gallery link and broken DOI badge Aug 13, 2026
@timtreis
timtreis merged commit 8bcfe6c into main Aug 13, 2026
8 checks passed
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.93%. Comparing base (fbb6872) to head (e628913).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #768      +/-   ##
==========================================
+ Coverage   79.66%   79.93%   +0.26%     
==========================================
  Files          18       18              
  Lines        4672     4814     +142     
  Branches     1036     1069      +33     
==========================================
+ Hits         3722     3848     +126     
- Misses        599      604       +5     
- Partials      351      362      +11     

see 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@timtreis
timtreis deleted the fix/notebooks-repo-rename branch August 13, 2026 17:53
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