-
Notifications
You must be signed in to change notification settings - Fork 391
Обновить tutorial-tic-tac-toe.md #952
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
@titovmx
titovmx
left a comment
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.
Спасибо за перевод! Оставил несколько комментариев
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.
предлагаю упростить
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.
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.
две строчки остались не переведены
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.
я не успел
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.
надо согласовать вводное предложение и пункты.
сейчас получается "Для этого вам необходимо скачайте Node.js"
@titovmx А как а как продолжать дальше переводить по новой все с начала? или ждать когда вы добавите перевод на сайт
Co-authored-by: Maxim Titov <mtitov92@gmail.com>
Co-authored-by: Maxim Titov <mtitov92@gmail.com>
Co-authored-by: Maxim Titov <mtitov92@gmail.com>
Co-authored-by: Maxim Titov <mtitov92@gmail.com>
Co-authored-by: Maxim Titov <mtitov92@gmail.com>
Co-authored-by: Maxim Titov <mtitov92@gmail.com>
Co-authored-by: Maxim Titov <mtitov92@gmail.com>
Co-authored-by: Maxim Titov <mtitov92@gmail.com>
Я дополнил перевод tutorial-tic-tac-toe и буду переводить дальше...
Обновить tutorial-tic-tac-toe.md
Дополнил перевод крестики-нолики
titovmx
commented
May 20, 2024
@SmaylikYoutuber привет! Обычно процесс такой:
- Вы выбираете статью для перевода и резервируете её за собой здесь. Также там ссылка на правила перевода, если ещё не видели, то, пожалуйста, ознакомьтесь с ними.
- Создаёте ветку в форкнутом репозитории, переводите и создаёте пулл реквест.
- Кто-то ревьюит, оставляет комменты, вы фиксите в этом же пулл реквесте.
- Аппрув, мерж, и переведённая статья попадает на сайт.
Если я правильно понял, сейчас вы правки внесли в новой ветке, но перевод ещё не закончен? Если так, то давайте закроем этот ПР и продолжим перевод там. Пока перевод не закончен, вы можете пометить пулл реквест как Черновик (Draft), чтобы статус был прозрачен для тех, кто будет проводить ревью.
SmaylikYoutuber
commented
Jun 13, 2024
@titovmx а можно по подробнее я сильно не разбираюсь в GitHub, я тут как новичок пришел просто помочь перевести, может какое-то видео подскажешь как продолжить перевод, а то я вообще не понял что было в верхней строчке. (Извини что не до конца понял и прошу написать по подробнее, что из за это-го я мучаю тебя, если ответишь буду рад)
SmaylikYoutuber
commented
Jun 13, 2024
вот может ссылку кинуть на новый перевод #953
Если ваш пулреквест является исправлением бага, а не переводом, то сперва убедитесь, что проблема относится ТОЛЬКО к https://ru.reactjs.org, а не к https://reactjs.org. Если это не так, то пулреквест следует открыть в родительском репозитории.