search

reactjs / ru.react.dev

React documentation website in Russian / Официальная русская версия сайта React
https://ru.react.dev
Creative Commons Attribution 4.0 International
678 stars 380 forks source link

Translate Refs and the DOM #236

Closed SmolinPavel closed 5 years ago

SmolinPavel commented 5 years ago

Translate Refs and the DOM

https://reactjs.org/docs/refs-and-the-dom.html

netlify[bot] commented 5 years ago

Deploy preview for ru-reactjs ready!

Built with commit f197849a1b40ff73c2a2686781845e2569f35a72

https://deploy-preview-236--ru-reactjs.netlify.com

another-guy commented 5 years ago

@SmolinPavel Ох, я не обратил внимание на то, что это draft. Но всё равно можете принять предложенные изменения прямо через GitHub. Это должно ускорить последующие ревью.

SmolinPavel commented 5 years ago

@SmolinPavel Ох, я не обратил внимание на то, что это draft. Но всё равно можете принять предложенные изменения прямо через GitHub. Это должно ускорить последующие ревью.

@another-guy Завтра планирую добить остатки и отполировать! В любом случае, спасибо за замечания, я их все обязательно учту 👍

SmolinPavel commented 5 years ago

@lex111, @another-guy Коллеги, я закончил работу над PR, но по каким-то причинам не могу перевести из draft в ready for review согласно документации. image

Нашёл следующую информацию по данному вопросу.

Могу создать новый PR или подождать решение проблемы. Буду ждать от вас рекомендаций! 👍

SmolinPavel commented 5 years ago

На всякий случай отправляю, как выглядит merge box у меня: image

lex111 commented 5 years ago

@SmolinPavel это странно, потому что @egorzekov тоже воспользовался этой новой возможностью гитхаба, и как я понял, у него кнопка видна, как собственно и мне (потому что я админ). Возможно, что-то блокирует эту кнопку? Расширение браузера? А если в другом браузере попробовать? Давайте попробуем разобраться. Всё-таки это может быть баг, тогда сможем помочь другим.

SmolinPavel commented 5 years ago

@SmolinPavel это странно, потому что @egorzekov тоже воспользовался этой новой возможностью гитхаба, и как я понял, у него кнопка видна, как собственно и мне (потому что я админ). Возможно, что-то блокирует эту кнопку? Расширение браузера? А если в другом браузере попробовать? Давайте попробуем разобраться. Всё-таки это может быть баг, тогда сможем помочь другим.

Попробовал в другом браузере - та же проблема :(

SmolinPavel commented 5 years ago

@another-guy @lex111 @ntishkevich - Большое спасибо за ревью! 👍

lex111 commented 5 years ago

@SmolinPavel попробуйте написать в саппорт гитхаба, он обычно оперативно отвечает. Просто странно, в чём баг.

SmolinPavel commented 5 years ago

@SmolinPavel попробуйте написать в саппорт гитхаба, он обычно оперативно отвечает. Просто странно, в чём баг.

@lex111 Проверил на другом проекте - у меня как у админа есть форма с классом branch-action-btn, которая отправляет пост запрос. На этой странице ничего нет. Да, сейчас напишу в саппорт

SmolinPavel commented 5 years ago

@another-guy Все правки внесены! В техподдержка GH уточнили, являюсь ли я автором PR, на что получили утвердительный ответ. Можно подождать, что они скажут, а можно вмержить и двигаться дальше 👍

gcor commented 5 years ago

@SmolinPavel, можно сделать пул-реквест не черновым, нажав "ready for review". Я сегодня ещё посмотрю текст.

SmolinPavel commented 5 years ago

@SmolinPavel, можно сделать пул-реквест не черновым, нажав "ready for review". Я сегодня ещё посмотрю текст.

@gcor Как раз с этим и проблема, т.к. у меня в мерж боксе нет такой кнопки (хотя я автор PR). Веду переписку с техподдержкой GH по данному вопросу

another-guy commented 5 years ago

@SmolinPavel учитывая, что ситуация затягивает ревью, я позволил себе отметить PR как ready for review и смёржить перевод. Спасибо оргомное за вклад!