Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: dto rationale #3987

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

docs: dto rationale #3987

wants to merge 1 commit into from

Conversation

euri10
Copy link
Contributor

@euri10 euri10 commented Feb 8, 2025

Description

  • add a topic on dto rationale

Closes

@euri10 euri10 requested review from a team as code owners February 8, 2025 17:36
@github-actions github-actions bot added area/docs This PR involves changes to the documentation size: small type/docs pr/external Triage Required 🏥 This requires triage labels Feb 8, 2025
DTO rationale
=============

.. admonition:: What particular problem does DTO solve in Litestar ?
Copy link
Member

Choose a reason for hiding this comment

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

Maybe make this a section in the "Basic Use" doc of DTO? Maybe at the bottom with a link somewhere here:
Снимок экрана 2025-02-08 в 22 24 22

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree it's good to know the why even if most people want the how !

I was looking at your suggestion, whcih I like, but I find 0-basic-use.html is already quite long and focued on the how, what about linking this dto-rationale just above in the index just before the content ?

20250209_0826_1081x661_1739085972

to something like :

20250209_0831_1041x539_1739086275

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs This PR involves changes to the documentation pr/external size: small Triage Required 🏥 This requires triage type/docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants