Skip to content
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

Lifting state up translated to Polish #11

Merged
merged 2 commits into from Feb 17, 2019

Conversation

iamarek
Copy link
Contributor

@iamarek iamarek commented Feb 14, 2019

Na pewno kilka kwestii do ustalenia 🙈

  • input (tłumaczymy?)
  • boilerplate (jak go użyć w tym kontekście)
  • tłumaczenie lifting state up

@jakubdrozdek
Copy link
Member

  • input = dane wejściowe
  • boilerplate: na razie zostawiłbym w oryginale, w cudzysłowie
  • lifting state up = wynoszenie stanu w górę drzewa?

@iamarek
Copy link
Contributor Author

iamarek commented Feb 14, 2019

@jakubdrozdek to tak w zasadzie lifting przetłumaczyłem, co do inputa, to chodzi mi o input jako tag HTMLowy, zakładam, że tego nie tłumaczymy, bo byłoby dziwnie, ale wole się upewnić.

@jakubdrozdek
Copy link
Member

Nie, znaczników nie tłumaczymy, tak jak nie tłumaczymy zmiennych i niektórych innych rzeczy w kodzie.

@jakubdrozdek jakubdrozdek added 💪 long page - more than 400 lines Long page needs review A pull request ready to be reviewed needs +1 Needs another approval before merging labels Feb 14, 2019
@jakubdrozdek jakubdrozdek added this to the Core Pages milestone Feb 15, 2019
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.

Przejrzałem całość ale @jakubdrozdek zrobił dobrą robotę więc nic więcej nie znalazłem. Ze swojej strony daje approve. 👍

@jakubdrozdek
Copy link
Member

To teraz czekamy na poprawki od @iamarek i można mergować :-)

@jakubdrozdek jakubdrozdek merged commit 4c3565d into reactjs:master Feb 17, 2019
@jakubdrozdek jakubdrozdek added 🌐 translated Translation is completed for this page and removed needs +1 Needs another approval before merging needs review A pull request ready to be reviewed labels Feb 17, 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.

None yet

3 participants