Skip to content

Translated 'Accessibility' docs page. #61

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Apr 9, 2019
Merged

Translated 'Accessibility' docs page. #61

merged 5 commits into from
Apr 9, 2019

Conversation

danzawadzki
Copy link
Contributor

@danzawadzki danzawadzki commented Mar 25, 2019 •
edited
Loading

Hey 👋,
This is my first contribution to this repository. I tried to keep your proposed convention of translating individual proper names.

Live podgląd

I will be grateful for the reviews 🙏! I would like to correct this page first, before I start the next one.

@danzawadzki danzawadzki requested a review from a team March 25, 2019 13:20
@netlify
Copy link

netlify bot commented Mar 25, 2019 •
edited
Loading

Deploy preview for pl-reactjs ready!

Built with commit 4821ec9

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

@cvqprs cvqprs reopened this Mar 26, 2019
@cvqprs cvqprs added the work in progress Author opened the pull-request but it's still being worked on label Mar 26, 2019
@danzawadzki
Copy link
Contributor Author

@cvqprs Thank you again for your help! 💪 Could you please change this pull request status to a ready to review?

@cvqprs cvqprs added needs review A pull request ready to be reviewed and removed work in progress Author opened the pull-request but it's still being worked on labels Mar 27, 2019
jakubdrozdek
jakubdrozdek requested changes Mar 27, 2019
View reviewed changes
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.

Dobre pierwsze tłumaczenie :-) Nie przejmuj się, że jest tak dużo sugestii - zwykle są to drobne błędy. Ogólnie czytało się przyjemnie.

@cvqprs cvqprs added needs author response Changes requested needing author's input and removed needs review A pull request ready to be reviewed labels Mar 28, 2019
@tesseralis tesseralis mentioned this pull request Apr 2, 2019
Closed
94 tasks
@danzawadzki
Copy link
Contributor Author

@jakubdrozdek Dzięki, że poświęciłeś tyle czasu na poprawki! Chyba mam rekord na sam początek 😂, ale rozumiem. Dobrze, żeby całość była napisana spójnym językiem, więc pewnie chwilkę zajmie mi załapanie.

Przepraszam, że po takim czasie, ale wypadł mi urlop. Zaraz po powrocie zmerguje poprawki!

@danzawadzki
Copy link
Contributor Author

@jakubdrozdek Mój pierwszy PR i trochę tu nakręciłem. Nie ogarnąłem opcji z batch-owym dodawaniem sugestii 😅. Ostatecznie zrobiłem rebase ze squash-em wszystkich commitów do jednego, stąd ten force push. Mam nadzieję, że nie pominąłem niczego. Trochę jeszcze błądzę, bo przyzwyczaiłem się do BB.

Mógłbyś prosze rzucić okiem, czy nie pominąłem żadnych zmian? Zrobiłem jeden duży commit ze wszystkimi sugestiami, ale z jakiegoś powodu wciąż duża cześć zmian nie jest rozwiązanych (mimo, że widze je w ostatnim commit-cie).

@jakubdrozdek
Copy link
Member

@danzawadzki Przeoczyłeś jedną, drobną zmianę ;-) Popraw proszę i będzie approve ode mnie. Wtedy już tylko "one to go".

@danzawadzki
Copy link
Contributor Author

danzawadzki commented Apr 8, 2019 •
edited
Loading

@jakubdrozdek @cvqprs Poszła ostatnia zmiana :). Przepraszam za pominięcie i dorzucenie więcej pracy 😞. Według mnie, wygląda już dobrze, ale czekam na Wasze review.

@cvqprs
Copy link
Member

cvqprs commented Apr 9, 2019

Dla mnie bomba! Czekamy na +1 i merge

@cvqprs cvqprs added needs +1 Needs another approval before merging and removed needs author response Changes requested needing author's input labels Apr 9, 2019
bartlomiejzuber
bartlomiejzuber approved these changes Apr 9, 2019
View reviewed changes
Copy link
Member

@bartlomiejzuber bartlomiejzuber left a comment

Choose a reason for hiding this comment

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

👍

@cvqprs cvqprs merged commit aaa1c7f into reactjs:master Apr 9, 2019
@cvqprs cvqprs added 🌐 translated Translation is completed for this page and removed needs +1 Needs another approval before merging labels Apr 10, 2019
@jakubdrozdek jakubdrozdek added this to the Other Pages milestone Apr 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jakubdrozdek jakubdrozdek jakubdrozdek requested changes

@cvqprs cvqprs cvqprs approved these changes

@bartlomiejzuber bartlomiejzuber bartlomiejzuber approved these changes

Assignees

@danzawadzki danzawadzki

Labels
🌐 translated Translation is completed for this page
Projects
None yet
Milestone
Other Pages
Development

Successfully merging this pull request may close these issues.
4 participants
@danzawadzki @jakubdrozdek @cvqprs @bartlomiejzuber