Skip to content

Conversation

@ericholscher
Copy link
Member

Summary

  • document the spike to restore /page/ redirects on subprojects
  • make ServePageRedirect resolve subprojects from either the current project, alias, or slug
  • add a regression test covering /projects/<subproject>/page/ redirects
  • document how to run the subproject page-redirect test with the proxito settings
  • fix a ruff lint warning in ServePageRedirect by removing an unused import

Testing

  • DJANGO_SETTINGS_MODULE=readthedocs.settings.proxito.test uv run pytest readthedocs/proxito/tests/test_old_redirects.py::InternalRedirectTests::test_page_redirect_on_subproject -vv -s
  • uv run ruff check readthedocs/proxito/views/serve.py readthedocs/proxito/tests/test_old_redirects.py

Codex Task

@ericholscher ericholscher requested review from a team as code owners December 2, 2025 20:05
@ericholscher ericholscher requested review from stsewd and removed request for agjohnson December 2, 2025 20:05
@read-the-docs-community
Copy link

read-the-docs-community bot commented Dec 2, 2025

Documentation build overview

📚 dev | 🛠️ Build #30541339 | 📁 Comparing 58191a4 against latest (1292c9f)


🔍 Preview build

Show files changed (6 files in total): 📝 6 modified | ➕ 0 added | ➖ 0 deleted
File Status
contribute.html 📝 modified
install.html 📝 modified
search-integration.html 📝 modified
style-guide.html 📝 modified
design/embed-api.html 📝 modified
design/theme-context.html 📝 modified

@read-the-docs-community
Copy link

read-the-docs-community bot commented Dec 2, 2025

Documentation build overview

📚 docs | 🛠️ Build #30541340 | 📁 Comparing 58191a4 against latest (1292c9f)


🔍 Preview build

Show files changed (20 files in total): 📝 17 modified | ➕ 0 added | ➖ 3 deleted
File Status
build-customization.html 📝 modified
link-previews.html 📝 modified
support.html 📝 modified
visual-diff.html 📝 modified
about/index.html 📝 modified
config-file/v2.html 📝 modified
intro/antora.html 📝 modified
intro/doctools.html 📝 modified
intro/docusaurus.html 📝 modified
intro/markdoc.html 📝 modified
intro/mdbook.html 📝 modified
intro/mkdocs.html 📝 modified
intro/mystmd.html ➖ deleted
intro/sphinx.html 📝 modified
intro/vitepress.html 📝 modified
intro/zensical.html ➖ deleted
reference/main-content-detection.html ➖ deleted
server-side-search/index.html 📝 modified
tutorial/index.html 📝 modified
tutorials/index.html 📝 modified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants