-
Notifications
You must be signed in to change notification settings - Fork 197
feat: add react query example #673
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
feat: add react query example #673
Conversation
✅ Deploy Preview for pr-fsd ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Спасибо, что оформил ПРчик! Русская версия выглядит хорошо, как мы, в целом, уже раньше и обсуждали, только местами есть опечаточки (типа "Полезный ссылки").
С английской версией немного сложнее :D В русской версии местами немного сложная структура предложений, и после перевода на английский смысл становится очень трудно уловить. Самый яркий пример — раздел CodeGeneration (я бы, кстати, сделал два слова, не сливал в одно).
There are tools for automatic code generation that are less flexible than those which can be configured as described above.
Это предложение предлагаю переписать вот так:
There are tools that can generate API code for you, but they are less flexible than the manual approach described above.
И в целом, предлагаю скормить этот текст какой-нибудь хорошей LLM-ке и попросить исправить ошибки и упростить язык, а потом проверить через LanguageTool (англ) и Главред (рус)
i18n/en/docusaurus-plugin-content-docs/current/guides/tech/with-react-query.mdx
Outdated
Show resolved
Hide resolved
i18n/en/docusaurus-plugin-content-docs/current/guides/tech/with-react-query.mdx
Outdated
Show resolved
Hide resolved
058535f
to
9e0315b
Compare
9e0315b
to
9300cdf
Compare
This PR added example of usage React Query with FSD