Skip to content

Add main preCICE API guide to documentation navigation - #261

Merged
MakisH merged 2 commits into
precice:developfrom
Shubham-Padkonde:docs/project-api-sidebar-link
Oct 1, 2026
Merged

MakisH merged 2 commits into
precice:developfrom
Shubham-Padkonde:docs/project-api-sidebar-link

Conversation

@Shubham-Padkonde

Copy link
Copy Markdown
Contributor

Fixes #258.

Add a preCICE navigation section linking to the main coupling API overview, so readers arriving directly at the Python bindings reference can find the broader project documentation from the sidebar.

Validation: built a navigation-only Sphinx fixture using the changed index and the repository's Read the Docs theme with warnings treated as errors; verified the link is present on the index and all three child pages. The target URL loads successfully and git diff --check passes. The full autodoc build was not run because the compiled preCICE bindings are not installed locally.

Prepared with Codex assistance.

@IshaanDesai
IshaanDesai requested a review from MakisH September 30, 2026 05:50
@MakisH

MakisH commented Sep 30, 2026

Copy link
Copy Markdown
Member

@Shubham-Padkonde thank you for the fix! To make reviewing easier, could you please add a screenshot of how this looks like?

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The focused documentation change correctly addresses the linked issue without introducing unresolved concerns.

Review effort: Balanced
Findings: None

What changed in this PR

Adds a sidebar link from the Python bindings documentation to the main preCICE coupling API guide, resolving #258.

Changes:

  • Adds a dedicated preCICE navigation section.
  • Links to the external coupling API overview.
File Description
docs/​index.rst Adds the coupling API overview to documentation navigation.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Shubham-Padkonde

Copy link
Copy Markdown
Contributor Author
precice261-sidebar

Comment thread docs/index.rst Outdated
@MakisH
MakisH merged commit c712c01 into precice:develop Oct 1, 2026
5 checks passed
@MakisH

MakisH commented Oct 1, 2026

Copy link
Copy Markdown
Member

I triggered the workflow; this is now rendered in https://api.precice.org/python/develop/

It will be brought to https://api.precice.org/python/latest/ in the next release.

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.

Sphinx documentation: No backlink to the main project

3 participants