Skip to content

Translate 'Getting Started' page to Polish #6

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 12 commits into from
Feb 18, 2019

Conversation

iamarek
Copy link
Contributor

@iamarek iamarek commented Feb 11, 2019

Getting Started page translated to Polish.

jakubdrozdek
jakubdrozdek previously approved these changes Feb 12, 2019
@iamarek
Copy link
Contributor Author

iamarek commented Feb 12, 2019

Should be ready to merge now.

@jakubdrozdek
Copy link
Member

Waiting for the other maintainers to approve. We need at least 2 approvals before merging. We don't want biased opinions. What's more, others can spot issues I didn't see and/or discuss about particular ways of translating things, which will be later added to the glossary.

cvqprs
cvqprs previously approved these changes Feb 12, 2019
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.

Dobra robota! Gratuluje pierwszego PR i merge

jakubdrozdek
jakubdrozdek previously approved these changes Feb 13, 2019
TeoTN
TeoTN previously requested changes Feb 13, 2019
jakubdrozdek
jakubdrozdek previously approved these changes Feb 14, 2019
@jakubdrozdek jakubdrozdek added the needs +1 Needs another approval before merging label Feb 14, 2019
@jakubdrozdek
Copy link
Member

@TeoTN @bartlomiejzuber @marcinostrouch Macie jeszcze jakieś uwagi? Jeśli nie, to dajcie approve, bo się kolejka robi...

marcinostrouch
marcinostrouch previously approved these changes Feb 17, 2019
Copy link
Contributor

@marcinostrouch marcinostrouch left a comment

Choose a reason for hiding this comment

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

Wydaje mi się, że z tymi kilkoma poprawkami doświadczenie czytelnicze będzie jeszcze lepsze.

@iamarek iamarek dismissed stale reviews from marcinostrouch and jakubdrozdek via 774031e February 17, 2019 17:40
Copy link
Contributor

@marcinostrouch marcinostrouch left a comment

Choose a reason for hiding this comment

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

OK

@jakubdrozdek jakubdrozdek dismissed stale reviews from TeoTN and bartlomiejzuber February 18, 2019 07:35

Fixed

@jakubdrozdek jakubdrozdek merged commit 8343c14 into reactjs:master Feb 18, 2019
@jakubdrozdek jakubdrozdek added 🌐 translated Translation is completed for this page and removed needs +1 Needs another approval before merging labels Feb 28, 2019
@jakubdrozdek jakubdrozdek added this to the Other Pages milestone Feb 28, 2019
@cvqprs cvqprs mentioned this pull request Mar 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌐 translated Translation is completed for this page
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants