Skip to content

Conversation

@Wiltzsu
Copy link
Contributor

@Wiltzsu Wiltzsu commented Oct 19, 2025

Q A
Bug fix? yes
New feature? no
Docs? yes
Issues Fixes #785
License MIT

Description

This pull request fixes broken documentation links in various README files that currently point to non-existent paths.

Updated all documentation links to point to the correct location: ../../docs/index.rst (the main documentation entry point).

Files modified:

  • src/agent/README.md
  • src/chat/README.md
  • src/platform/README.md
  • src/store/README.md
  • src/ai-bundle/README.md
  • src/mcp-bundle/README.md

Change type: Simple find-and-replace of broken relative paths

@carsonbot carsonbot added Bug Something isn't working Status: Needs Review labels Oct 19, 2025
@carsonbot carsonbot changed the title Docs/fix readme links Docs/fix readme links Oct 19, 2025
@chr-hertel
Copy link
Member

Hi @Wiltzsu, thanks for fixing that and solving the issue!

I have a conceptual question here, more about #749 and the other restructurings by @OskarStark - what was your idea with the subtree split here? We're running in an inconsistency with the README links here now - or is that still to solve?

@OskarStark
Copy link
Contributor

My plan is to focus on the subtree split and go with the rest files as primary doc source

@chr-hertel
Copy link
Member

What I meant is that with the change here the readme of the splitted component would be broken, e.g. https://github.com/symfony/ai-chat

@chr-hertel
Copy link
Member

anyhow, this is less broken now - and we'll have a deployment of the docs we could reference there instead 👍

@chr-hertel chr-hertel force-pushed the docs/fix-readme-links branch from 0563787 to bf5e965 Compare October 20, 2025 19:00
@chr-hertel
Copy link
Member

Thank you @Wiltzsu.

@chr-hertel chr-hertel merged commit f3a1513 into symfony:main Oct 20, 2025
1 check passed
@Wiltzsu
Copy link
Contributor Author

Wiltzsu commented Oct 20, 2025

Thank you @Wiltzsu.

Happy to help!

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

Labels

Bug Something isn't working Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs links in READMEs are broken

4 participants