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

Обновить tutorial-tic-tac-toe.md #952

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

SmaylikYoutuber
Copy link

Если ваш пулреквест является исправлением бага, а не переводом, то сперва убедитесь, что проблема относится ТОЛЬКО к https://ru.reactjs.org, а не к https://reactjs.org. Если это не так, то пулреквест следует открыть в родительском репозитории.

Copy link

vercel bot commented May 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
ru-legacy-reactjs-org ⬜️ Ignored (Inspect) Visit Preview May 12, 2024 7:37am

Copy link
Collaborator

@titovmx titovmx left a comment

Choose a reason for hiding this comment

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

Спасибо за перевод! Оставил несколько комментариев

src/content/learn/tutorial-tic-tac-toe.md Outdated Show resolved Hide resolved
src/content/learn/tutorial-tic-tac-toe.md Outdated Show resolved Hide resolved
src/content/learn/tutorial-tic-tac-toe.md Outdated Show resolved Hide resolved
src/content/learn/tutorial-tic-tac-toe.md Outdated Show resolved Hide resolved

You can see what it will look like when you're finished here:
Вы можете увидеть, как это будет выглядеть, когда вы закончите здесь:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Вы можете увидеть, как это будет выглядеть, когда вы закончите здесь:
Вы можете увидеть ожидаемый результат здесь:

предлагаю упростить

@@ -57,9 +57,9 @@ function Board({ xIsNext, squares, onPlay }) {
const winner = calculateWinner(squares);
let status;
if (winner) {
status = 'Winner: ' + winner;
status = 'Выйграл: ' + winner;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
status = 'Выйграл: ' + winner;
status = 'Победитель: ' + winner;

src/content/learn/tutorial-tic-tac-toe.md Outdated Show resolved Hide resolved
src/content/learn/tutorial-tic-tac-toe.md Outdated Show resolved Hide resolved
1. Unzip the archive, then open a terminal and `cd` to the directory you unzipped
1. Скачайте [Node.js](https://nodejs.org/en/)
1. На вкладке CodeSandbox, которую вы открыли ранее, нажмите кнопку в левом верхнем углу, чтобы открыть меню, а затем выберите Скачать Sandbox в этом меню, чтобы загрузить архив файлов локально.
1. Разархивируйте архив, затем откройте терминал и перейдите в каталог, который вы разархивировали.
1. Install the dependencies with `npm install`
Copy link
Collaborator

Choose a reason for hiding this comment

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

две строчки остались не переведены

Copy link
Author

Choose a reason for hiding this comment

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

я не успел

1. Install [Node.js](https://nodejs.org/en/)
1. In the CodeSandbox tab you opened earlier, press the top-left corner button to open the menu, and then choose **Download Sandbox** in that menu to download an archive of the files locally
1. Unzip the archive, then open a terminal and `cd` to the directory you unzipped
1. Скачайте [Node.js](https://nodejs.org/en/)
Copy link
Collaborator

Choose a reason for hiding this comment

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

надо согласовать вводное предложение и пункты.
сейчас получается "Для этого вам необходимо скачайте Node.js"

@SmaylikYoutuber SmaylikYoutuber deleted the patch-1 branch May 11, 2024 06:30
@SmaylikYoutuber SmaylikYoutuber restored the patch-1 branch May 11, 2024 06:30
@SmaylikYoutuber
Copy link
Author

SmaylikYoutuber commented May 11, 2024

@titovmx А как а как продолжать дальше переводить по новой все с начала? или ждать когда вы добавите перевод на сайт

SmaylikYoutuber and others added 8 commits May 12, 2024 08:57
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 и буду переводить дальше...
Дополнил перевод крестики-нолики
@titovmx
Copy link
Collaborator

titovmx commented May 20, 2024

@SmaylikYoutuber привет! Обычно процесс такой:

  1. Вы выбираете статью для перевода и резервируете её за собой здесь. Также там ссылка на правила перевода, если ещё не видели, то, пожалуйста, ознакомьтесь с ними.
  2. Создаёте ветку в форкнутом репозитории, переводите и создаёте пулл реквест.
  3. Кто-то ревьюит, оставляет комменты, вы фиксите в этом же пулл реквесте.
  4. Аппрув, мерж, и переведённая статья попадает на сайт.

Если я правильно понял, сейчас вы правки внесли в новой ветке, но перевод ещё не закончен? Если так, то давайте закроем этот ПР и продолжим перевод там. Пока перевод не закончен, вы можете пометить пулл реквест как Черновик (Draft), чтобы статус был прозрачен для тех, кто будет проводить ревью.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants