Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

🌐 Update Chinese translation fordocs/zh/docs/tutorial/dependencies/dependencies-with-yield.md#12028

Merged
alejsdev merged 6 commits intofastapi:masterfrom
xuvjso:dependencies-with-yield-docs-fix
Aug 19, 2024
Merged

🌐 Update Chinese translation fordocs/zh/docs/tutorial/dependencies/dependencies-with-yield.md#12028
alejsdev merged 6 commits intofastapi:masterfrom
xuvjso:dependencies-with-yield-docs-fix

Conversation

@xuvjso
Copy link
Contributor

#10831 ensures that the code followingyield executes after the response is create but before it is sent to the client. The statement in#3620 (comment) was outdated, it is now safe to commit the session afteryield as the code is guaranteed to execute after the response is created but before it is sent.

defget_db()->Session:session=SessionLocal()try:yieldsessionsession.commit()except:session.rollback()raisefinally:session.close()

@github-actionsgithub-actionsbot added the docsDocumentation about how to use FastAPI labelAug 16, 2024
@estebanx64estebanx64 added lang-zhChinese translations awaiting-review labelsAug 16, 2024
@estebanx64estebanx64 changed the titleUpdate docs about dependencies with yield🌐 Update docs about dependencies with yieldAug 16, 2024
@estebanx64
Copy link
Contributor

Thanks for your time and interest in helping us! 🙇

We are going to review these little changes 🤓

@estebanx64estebanx64 added lang-allTranslations and removed docsDocumentation about how to use FastAPI labelsAug 16, 2024
@github-actionsgithub-actionsbot added the docsDocumentation about how to use FastAPI labelAug 16, 2024
@xuvjso
Copy link
ContributorAuthor

I noticed that the chinese version of the "dependencies with yield" section of the documentation is outdated, so I provided an updated translation to prevent users from being misled by outdated information and to align with the latest English documentation.

@alejsdevalejsdev removed the docsDocumentation about how to use FastAPI labelAug 19, 2024
@alejsdev
Copy link
Member

Hi@xuvjso
Thanks for your interest in contributing to FastAPI 🙇‍♀️
To avoid problems with the labels. Could you better separate the EN from the ZH docs in another PR? Thanks in advance! 🙏

@github-actionsgithub-actionsbot added the docsDocumentation about how to use FastAPI labelAug 19, 2024
@github-actions
Copy link
Contributor

@alejsdevalejsdev removed the docsDocumentation about how to use FastAPI labelAug 19, 2024
@alejsdevalejsdev merged commitf0866bc intofastapi:masterAug 19, 2024
@alejsdev
Copy link
Member

alejsdev commentedAug 19, 2024
edited
Loading

Great! Thanks for your contribution 🥳@xuvjso
And thanks for the review 💯@Aruelius

@alejsdevalejsdev changed the title🌐 Update docs about dependencies with yield🌐 Update Chinese translation fordocs/zh/docs/tutorial/dependencies/dependencies-with-yield.mdAug 19, 2024
@xuvjso
Copy link
ContributorAuthor

Great! Thanks for your contribution 🥳@xuvjso And thanks for the review 💯@Aruelius

It's my pleasure. 😊

@xuvjsoxuvjso deleted the dependencies-with-yield-docs-fix branchAugust 19, 2024 17:50
black-redoc pushed a commit to black-redoc/fastapi that referenced this pull requestSep 12, 2024
black-redoc pushed a commit to black-redoc/fastapi that referenced this pull requestSep 12, 2024
black-redoc pushed a commit to black-redoc/fastapi that referenced this pull requestSep 12, 2024
s-rigaud pushed a commit to s-rigaud/fastapi that referenced this pull requestJan 23, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

1 more reviewer

@ArueliusArueliusAruelius approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved-1lang-allTranslationslang-zhChinese translations

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@xuvjso@estebanx64@alejsdev@Aruelius

Comments


[8]ページ先頭

©2009-2026 Movatter.jp