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/how-to/index.md#12070

Merged
alejsdev merged 3 commits intofastapi:masterfrom
synthpop123:master
Aug 28, 2024
Merged

🌐 Update Chinese translation fordocs/zh/docs/how-to/index.md#12070
alejsdev merged 3 commits intofastapi:masterfrom
synthpop123:master

Conversation

@synthpop123
Copy link
Contributor

The小技巧 label in the admonition was causing rendering issues in MkDocs, so replacing it with the standardtip label fixes this problem.

@github-actionsgithub-actionsbot added the lang-allTranslations labelAug 25, 2024
@github-actions
Copy link
Contributor

@alejsdevalejsdev added lang-zhChinese translations awaiting-review labelsAug 25, 2024
@alejsdev
Copy link
Member

Hi@synthpop123, you're right! This is an error, but we usually prefer to keep the original (for the styling) and the translation, so it would be something more like this:

/// tip | translated-word

Could you update it, please? 😄

Thank you 🙇‍♀️

@github-actions
Copy link
Contributor

@synthpop123
Copy link
ContributorAuthor

Hi@synthpop123, you're right! This is an error, but we usually prefer to keep the original (for the styling) and the translation, so it would be something more like this:

/// tip | translated-word

Could you update it, please? 😄

Thank you 🙇‍♀️

Thank you for the quick review! I’ve updated the PR with the suggested change.

alejsdev reacted with rocket emoji

@github-actions
Copy link
Contributor

Copy link
Member

@svlandegsvlandeg left a comment

Choose a reason for hiding this comment

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

Nice catch, thanks! 🙏

alejsdev reacted with hooray emoji
@alejsdevalejsdev merged commitf41f623 intofastapi:masterAug 28, 2024
@alejsdev
Copy link
Member

Thank you!@synthpop123@svlandeg 🎉

svlandeg and synthpop123 reacted with hooray emoji

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

@svlandegsvlandegsvlandeg approved these changes

+1 more reviewer

@ArueliusArueliusAruelius approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved-2lang-allTranslationslang-zhChinese translations

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@synthpop123@alejsdev@svlandeg@Aruelius

Comments


[8]ページ先頭

©2009-2026 Movatter.jp