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

🌐 Add Russian translation for docs/tutorial/extra-models.md #9619

Merged
merged 3 commits into from Jun 22, 2023

Conversation

ivan-abc
Copy link
Contributor

@ivan-abc ivan-abc commented Jun 5, 2023

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 5, 2023

📝 Docs preview for commit 5c0ee87 at: https://647dee0f244d63103214aeb9--fastapi.netlify.app

* **Модель для вывода** не должна содержать пароль.
* **Модель для базы данных**, возможно, должна содержать хэшированный пароль.

!!! danger "Внимание"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
!!! danger "Внимание"
!!! danger Внимание

Copy link
Contributor

Choose a reason for hiding this comment

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

Мы до этого всегда писали без скобок, думаю есть смысл поддерживать стиль :)

Co-authored-by: Alexandr <alexandrhub@vk.com>
@github-actions
Copy link
Contributor

github-actions bot commented Jun 6, 2023

📝 Docs preview for commit 81de89e at: https://647f832f3c6003644e9f2c2d--fastapi.netlify.app

@ivan-abc
Copy link
Contributor Author

ivan-abc commented Jun 9, 2023

Всё-таки кавычки тут играют роль, т.к. первое ключевое слово (danger) распознаётся как тип "плашки" (заметка, предупреждение, инфо...), а второе - как её заголовок. Но если второе не в кавычках, то при рендере документации оно игнорируется и подставляется дефолтный английский заголовок. А если писать только русский заголовок, то не распознаётся тип плашки и они все рендерятся одинаковыми.

Например, так отображается !!! warning "Внимание":
image

А так !!! warning Внимание (второе слово игнорируется):
image

А так !!! Внимание (без warning - тогда не распознаётся тип плашки):
image

@github-actions
Copy link
Contributor

github-actions bot commented Jun 9, 2023

📝 Docs preview for commit c3d837c at: https://6482e275d203433a8e25ddee--fastapi.netlify.app

@Kludex Kludex added lang-all Translations lang-ru Russian translations awaiting-review labels Jun 10, 2023
@Xewus
Copy link
Contributor

Xewus commented Jun 11, 2023

Very good!

@tiangolo
Copy link
Owner

Thanks @ivan-abc! 🚀

And thanks for the reviews @Xewus and @Alexandrhub

@tiangolo tiangolo merged commit 612cbee into tiangolo:master Jun 22, 2023
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-2 lang-all Translations lang-ru Russian translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants