Skip to content

Translate 'ReactDOM' page #35

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

Merged
merged 6 commits into from
Apr 8, 2019
Merged

Translate 'ReactDOM' page #35

merged 6 commits into from
Apr 8, 2019

Conversation

patpienkowski
Copy link
Contributor

Translations for ReactDOM page

Translations for ReactDOM page
@netlify
Copy link

netlify bot commented Mar 1, 2019

Deploy preview for pl-reactjs ready!

Built with commit e624fca

https://deploy-preview-35--pl-reactjs.netlify.com

Copy link
Member

@jakubdrozdek jakubdrozdek left a comment

Choose a reason for hiding this comment

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

@patpienkowski Cześć, witamy na pokładzie.
Przykro mi to mówić, ale niestety Twoje tłumaczenie nie jest najlepsze. Wygląda, jakbyś większość tekstu przepuścił przez translator i nawet go nie przeczytał. Dodałem sporo sugestii poprawek. Przeczytaj je proszę i dokończ tłumaczenie (nie poprawiłem jedynie kilku ostatnich akapitów).
Jak już skończysz, to dla pewności przeczytaj cały tekst raz jeszcze, żeby upewnić się, że wszystko jest spójne logicznie.
Dokumentacja jest jak książka i powinno się ją tak czytać. Jeśli pomiędzy zdaniami nie będzie logicznego połączenia, to czytelnik wyjdzie po chwili i nigdy nie wróci. A tego chcemy uniknąć :-)

@jakubdrozdek jakubdrozdek added needs author response Changes requested needing author's input needs review A pull request ready to be reviewed needs +1 Needs another approval before merging labels Mar 3, 2019
@jakubdrozdek jakubdrozdek added this to the Core Pages milestone Mar 3, 2019
@patpienkowski
Copy link
Contributor Author

@jakubdrozdek Masz racje. Przepraszam za takie tłumaczenie. W Polsce się nie urodziłem więc według mnie takie tłumaczenie brzmi hehe. Nie spodziewałem się ze tak zle będzie.
Dziękuję za twoją uczciwość. Potrzebuje więcej czasu żeby te tłumaczenie doprowadzić do lepszego stanu. Przepraszam jeszcze raz.

@jakubdrozdek
Copy link
Member

@patpienkowski Dzięki za zrozumienie ;-) Nie ma za co przepraszać. Czekamy na poprawki :-)

@jakubdrozdek
Copy link
Member

@patpienkowski Nie trzeba, możesz wrzucać wszystko do tego PR-a 😊

Requested changes added to reference-react-dom.md
@patpienkowski patpienkowski requested a review from a team March 20, 2019 22:10
tylna furtka - wyjscie awaryjne...
@patpienkowski
Copy link
Contributor Author

@jakubdrozdek Czesc! Dodalem poprawki pare dni temu. Nieiwiem czy zauwarzyles ale sa dodane jak bys mial czas przegladnac!

Copy link
Member

@jakubdrozdek jakubdrozdek left a comment

Choose a reason for hiding this comment

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

Pojawiły się nowe błędy.

@cvqprs cvqprs removed the needs review A pull request ready to be reviewed label Mar 26, 2019
@cvqprs
Copy link
Member

cvqprs commented Mar 31, 2019

Hej @patpienkowski daj znac prosze, czy bedziesz jeszcze pracowal nad tym PR. Jezlei nie, to moge go przejac, dodac poprawki etc.

@patpienkowski
Copy link
Contributor Author

patpienkowski commented Apr 1, 2019 via email

@cvqprs
Copy link
Member

cvqprs commented Apr 2, 2019

hej @patpienkowski prawie wszystko ok :-)

Czyta sie dobrze, wiec tutaj zero uwagi, jedynie formatowanie sie troche rozjechalo.

Porownaj sobie https://deploy-preview-35--pl-reactjs.netlify.com/docs/react-dom.html vs https://reactjs.org/docs/react-dom.html sekcje: Uwaga i findDOMNode

Trzeba naniesc drobne zmiany zeby poprawic wyglad wtedy dajemy +1 i merge, nie wiecej niz 10 minut pracy :-)

Copy link
Member

@cvqprs cvqprs left a comment

Choose a reason for hiding this comment

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

Super! Wszystko ok! Gratuluje pierwszego tlumaczenia! :-)

@cvqprs cvqprs dismissed bartlomiejzuber’s stale review April 8, 2019 05:34

Hej @bartlomiejzuber, @patpienkowski dodal zmiany zasugerowane przez @jakubdrozdek wiec zamykam ten request.

@cvqprs cvqprs merged commit 9d65636 into reactjs:master Apr 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs author response Changes requested needing author's input
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants