Skip to content

Conversation

@bokelley
Copy link
Contributor

@bokelley bokelley commented Nov 4, 2025

Fixes #676

Changes

  • ❌ Removed non-existent docs/api.md link from README Documentation section
  • ✅ Fixed docs/testing.md link to point to docs/testing/README.md (actual path)
  • 🧹 Removed duplicate Setup Guide link

Verification

All documentation links now point to existing files:

  • ✅ docs/SETUP.md
  • ✅ docs/DEVELOPMENT.md
  • ✅ docs/testing/README.md
  • ✅ docs/deployment.md
  • ✅ docs/TROUBLESHOOTING.md
  • ✅ docs/ARCHITECTURE.md

Testing

  • Pre-commit hooks: ✅ Passed
  • Unit tests: ✅ Passed (861 passed)
  • Integration tests: ✅ Passed (32 passed)
  • Integration v2 tests: ✅ Passed (28 passed)

- Removed non-existent docs/api.md link from README
- Fixed docs/testing.md link to point to docs/testing/README.md
- Removed duplicate Setup Guide link
- All documentation links now point to existing files

Fixes #676
@bokelley bokelley merged commit 9fd959e into main Nov 4, 2025
10 checks passed
danf-newton pushed a commit to Newton-Research-Inc/salesagent that referenced this pull request Nov 24, 2025
…otocol#700)

- Removed non-existent docs/api.md link from README
- Fixed docs/testing.md link to point to docs/testing/README.md
- Removed duplicate Setup Guide link
- All documentation links now point to existing files

Fixes adcontextprotocol#676
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.

The README Link to API Documentation is dead

2 participants