Skip to content

JSX in depth PL translation #181

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 3 commits into from
Mar 3, 2020

Conversation

michal-bednarz
Copy link
Contributor

Było kilka wątpliwości, przeczytajcie i sprawdźcie proszę czy jest ok :)

@michal-bednarz michal-bednarz requested a review from a team January 19, 2020 09:21
@netlify
Copy link

netlify bot commented Jan 19, 2020

Deploy preview for pl-reactjs ready!

Built with commit e4c1fe6

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

return <components[props.storyType] story={props.story} />;
}
```

To fix this, we will assign the type to a capitalized variable first:
Aby to naprawić przypisujemy najpierw wyrażenie do zmiennej z nazwą zaczynającą się z wielkiej litery:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Aby to naprawić przypisujemy najpierw wyrażenie do zmiennej z nazwą zaczynającą się z wielkiej litery:
Aby to naprawić, przypisujemy najpierw wyrażenie do zmiennej z nazwą zaczynającą się od wielkiej litery:

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Aby to naprawić przypisujemy najpierw wyrażenie do zmiennej z nazwą zaczynającą się z wielkiej litery:
Aby to naprawić przypisujemy najpierw wyrażenie do zmiennej z nazwą zaczynającą się wielką literą:

Copy link
Member

Choose a reason for hiding this comment

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

Myślę, że za dużo tu "ą" ("nazwą zaczynającą się wielką literą") i zostawiłbym moją wersję.

Copy link
Contributor

Choose a reason for hiding this comment

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

jadnak powinno być wielką literą lub od wielkiej litery
https://sjp.pwn.pl/poradnia/haslo/wielka-duza-litera-czy-z-wielkiej-duzej-litery;3927.html

@michal-bednarz
Copy link
Contributor Author

Dzisiaj zajmę się tym pull requestem

return <components[props.storyType] story={props.story} />;
}
```

To fix this, we will assign the type to a capitalized variable first:
Aby to naprawić przypisujemy najpierw wyrażenie do zmiennej z nazwą zaczynającą się z wielkiej litery:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Aby to naprawić przypisujemy najpierw wyrażenie do zmiennej z nazwą zaczynającą się z wielkiej litery:
Aby to naprawić przypisujemy najpierw wyrażenie do zmiennej z nazwą zaczynającą się wielką literą:

@jakubdrozdek
Copy link
Member

@stepek Fajnie, że wpadłeś :-) Nie chciałoby Ci się czegoś przetłumaczyć?

@jakubdrozdek jakubdrozdek added the needs author response Changes requested needing author's input label Jan 29, 2020
Co-Authored-By: Jakub Drozdek <30927218+jakubdrozdek@users.noreply.github.com>
Co-Authored-By: Kamil Stepczuk <5058678+stepek@users.noreply.github.com>
@jakubdrozdek
Copy link
Member

@PanFrontend Dorzuciłem kilka rzeczy, które umknęły Ci z poprzedniego review. Zwróć też uwagę, że dorzuciłem sugestie tłumaczenia tekstu w kodzie. Nie dodasz tego z poziomu WWW, więc zajrzyj do pliku i popraw :)

@michal-bednarz
Copy link
Contributor Author

Jasne, dzięki :) z założenia wprowadzałem tylko poprawki co do których miałem 100% pewności i robiłem to poprzez interfejs githuba zostawiając resztę (m.in kod właśnie) do poprawienia z poziomu edytora. Dam znać jak będzie gotowe :)

@stepek
Copy link
Contributor

stepek commented Feb 3, 2020

@stepek Fajnie, że wpadłeś :-) Nie chciałoby Ci się czegoś przetłumaczyć?

Spoko coś wezmę ;)

@michal-bednarz
Copy link
Contributor Author

Ok widzę już skąd tutaj zamieszanie, chciałem zastosować tylko część zmian z poziomu interfejsu github'a, a resztę wprowadzić i wypchnąć w drugim kroku. Natomiast github po commicie ze zmianami z inferfejsu przedawnił wszystkie wątki. Poprawię resztę na dniach

@michal-bednarz
Copy link
Contributor Author

@jakubdrozdek już powinno być ok

@jakubdrozdek jakubdrozdek added needs +1 Needs another approval before merging and removed needs author response Changes requested needing author's input labels Feb 20, 2020
@jakubdrozdek
Copy link
Member

@stepek Chciałbyś coś jeszcze dodać? Czy mergujemy?

@jakubdrozdek jakubdrozdek added 🌐 translated Translation is completed for this page and removed needs +1 Needs another approval before merging labels Mar 3, 2020
@jakubdrozdek jakubdrozdek merged commit f26ebff into reactjs:master Mar 3, 2020
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.

3 participants