Skip to content

Conversation

@jerome-white
Copy link
Contributor

@jerome-white jerome-white commented May 12, 2025

Summary

Target issue is #180

Add documentation for document and collection routes

Checklist

Before submitting a pull request, please ensure that you mark these task.

  • Ran fastapi run --reload app/main.py or docker compose up in the repository root and test.
  • If you've fixed a bug or added code that is tested and has test cases.

Notes

Documentation is structured in two ways:

  1. Parameter documentation is placed directly in the description tags. This goes for "simple" types in the router definition parameter list, and for "complex" types defined as classes composed of simple types. Notably, these descriptions are placed within the Python source files.
  2. Route documentation exists in standalone Markdown files. They are loaded via a custom function that knows where they live in the codebase.

@jerome-white jerome-white self-assigned this May 12, 2025
@jerome-white jerome-white added the documentation Improvements or additions to documentation label May 12, 2025
@jerome-white jerome-white linked an issue May 12, 2025 that may be closed by this pull request
@codecov
Copy link

codecov bot commented May 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@jerome-white jerome-white marked this pull request as ready for review May 14, 2025 17:12
@jerome-white jerome-white requested review from AkhileshNegi, avirajsingh7 and nishika26 and removed request for AkhileshNegi and avirajsingh7 May 14, 2025 17:13
@jerome-white jerome-white requested a review from AkhileshNegi May 20, 2025 03:38
@AkhileshNegi AkhileshNegi merged commit 5625315 into main May 23, 2025
1 check passed
@AkhileshNegi AkhileshNegi deleted the documentation/route-docs branch May 23, 2025 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Improve documentation for document and collection routes

4 participants