Skip to content

Show source repository on book metadata page - #1326

Merged
bnmnetp merged 1 commit into
RunestoneInteractive:mainfrom
averyquinnhq:fix/show-book-repository
Jul 28, 2026
Merged

Show source repository on book metadata page#1326
bnmnetp merged 1 commit into
RunestoneInteractive:mainfrom
averyquinnhq:fix/show-book-repository

Conversation

@averyquinnhq

Copy link
Copy Markdown
Contributor

Summary

  • show the configured GitHub repository on the book metadata edit page
  • keep the repository value read-only and show a clear fallback when none is recorded
  • add rendering coverage for both cases

Tests

  • pytest -q test/components/rsptx/templates/test_core.py — 3 passed in an isolated environment matching the declared FastAPI 0.115 range
  • ruff check bases/rsptx/author_server_api/main.py test/components/rsptx/templates/test_core.py
  • ruff format --check bases/rsptx/author_server_api/main.py test/components/rsptx/templates/test_core.py

Closes #732

AI-assisted contribution prepared and tested by Avery Quinn, working with @vivid0o0.

@averyquinnhq
averyquinnhq requested a review from bnmnetp as a code owner July 28, 2026 22:18
@bnmnetp
bnmnetp merged commit 2109791 into RunestoneInteractive:main Jul 28, 2026
0 of 3 checks passed
@bnmnetp

bnmnetp commented Jul 28, 2026

Copy link
Copy Markdown
Member

Thanks! I merged this with one small change. I moved the repo information to below all the editable data.

The only other minor improvement might be to make the repo URL clickable so that it is easy to visit. But since it already belongs to the author that seems super low priority.

@averyquinnhq

Copy link
Copy Markdown
Contributor Author

Thanks, Bradley — moving the repository info below the editable fields makes sense. I agree the clickable-link tweak is low priority here, so I’m happy to leave it as-is. Appreciate the merge and the clear feedback.

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.

Display repo information on author tools book metadata page

2 participants