Skip to content

Hooks reference #65

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 14 commits into from
Jun 2, 2019
Merged

Hooks reference #65

merged 14 commits into from
Jun 2, 2019

Conversation

g12i
Copy link
Contributor

@g12i g12i commented Mar 29, 2019

Tasks

  • Translate intro
  • Translate useState reference
  • Translate useEffect reference
  • Translate useContext reference
  • Translate useReducer reference
  • Translate useCallback reference
  • Translate useMemo reference
  • Translate useRef reference
  • Translate useImperativeHandle reference
  • Translate useLayoutEffect reference
  • Translate useDebugValue reference
  • Self review
  • Review

Preview

https://deploy-preview-65--pl-reactjs.netlify.com/docs/hooks-reference.html

@g12i g12i added the work in progress Author opened the pull-request but it's still being worked on label Mar 29, 2019
@netlify
Copy link

netlify bot commented Mar 29, 2019

Deploy preview for pl-reactjs ready!

Built with commit e4ddfa1

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

@jakubdrozdek
Copy link
Member

@g12i Pracujesz jeszcze nad tym?

@g12i
Copy link
Contributor Author

g12i commented May 7, 2019

@jakubdrozdek - tak, postaram się to dokończyć w najbliższym czasie.

@bartlomiejzuber
Copy link
Member

@g12i Jeżeli nie masz czasu mogę to dokończyć za Ciebie. :)

@g12i
Copy link
Contributor Author

g12i commented May 20, 2019

@bartlomiejzuber - 👍. W zasadzie miałem to przeczytać już tylko, czy o czymś nie zapomniałem i czy nie palnąłem jakiejś głupoty. Ale fakt - nie mam ostatnio czasu na to. Będę wdzięczny za pomoc i przepchnięcie dalej

@g12i
Copy link
Contributor Author

g12i commented May 20, 2019

@bartlomiejzuber - 👍. W zasadzie miałem to przeczytać już tylko, czy o czymś nie zapomniałem i czy nie palnąłem jakiejś głupoty. Ale fakt - nie mam ostatnio czasu na to, będę wdzięczny za pomoc i przepchnięcie dalej

@g12i g12i added the help wanted Extra attention is needed label May 20, 2019
@bartlomiejzuber
Copy link
Member

@g12i Sobotę mam wolną to przejrzę całość i dokończę co tam będzie trzeba. 👍

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.

Drobne poprawki, całościowo wszystko bardzo ładnie, dodaj sugestie lub odrzuć. Pozdrawiam :)

>
>If you want to run an effect and clean it up only once (on mount and unmount), you can pass an empty array (`[]`) as a second argument. This tells React that your effect doesn't depend on *any* values from props or state, so it never needs to re-run. This isn't handled as a special case -- it follows directly from how the dependencies array always works.
>Jeśli chcesz przeprowadzić efekt i posprzątać po nim tylko raz (podczas montowania i odmontowania), możesz przekazać pustą tablicę (`[]`) jako drugi argument. Dzięki temu React wie, że twój efekt nie zależy od *jakichkolwiek* wartości właściwości lub stanu, więc nigdy nie musi być ponownie uruchamiany. Nie jest to traktowane jako przypadek specjalny -- wynika to bezpośrednio z tego, jak zawsze działa tablica wejść.
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
>Jeśli chcesz przeprowadzić efekt i posprzątać po nim tylko raz (podczas montowania i odmontowania), możesz przekazać pustą tablicę (`[]`) jako drugi argument. Dzięki temu React wie, że twój efekt nie zależy od *jakichkolwiek* wartości właściwości lub stanu, więc nigdy nie musi być ponownie uruchamiany. Nie jest to traktowane jako przypadek specjalny -- wynika to bezpośrednio z tego, jak zawsze działa tablica wejść.
>Jeśli chcesz uruchomić i posprzątać efekt tylko raz (podczas montowania i odmontowania), możesz przekazać pustą tablicę (`[]`) jako drugi argument. Dzięki temu React wie, że twój efekt nie zależy od *jakichkolwiek* wartości właściwości lub stanu, więc nigdy nie musi być ponownie uruchamiany. Nie jest to traktowane jako przypadek specjalny -- wynika to bezpośrednio z tego, jak zawsze działa tablica wejść.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm, a tak:

Suggested change
>Jeśli chcesz przeprowadzić efekt i posprzątać po nim tylko raz (podczas montowania i odmontowania), możesz przekazać pustą tablicę (`[]`) jako drugi argument. Dzięki temu React wie, że twój efekt nie zależy od *jakichkolwiek* wartości właściwości lub stanu, więc nigdy nie musi być ponownie uruchamiany. Nie jest to traktowane jako przypadek specjalny -- wynika to bezpośrednio z tego, jak zawsze działa tablica wejść.
>Jeśli chcesz uruchomić efekt i posprzątać po nim tylko raz (podczas montowania i odmontowania), możesz przekazać pustą tablicę (`[]`) jako drugi argument. Dzięki temu React wie, że twój efekt nie zależy od *jakichkolwiek* wartości właściwości lub stanu, więc nigdy nie musi być ponownie uruchamiany. Nie jest to traktowane jako przypadek specjalny -- wynika to bezpośrednio z tego, jak zawsze działa tablica wejść.

g12i and others added 2 commits May 30, 2019 19:46
Co-Authored-By: Bartłomiej Żuber <[email protected]>
@g12i g12i marked this pull request as ready for review May 30, 2019 17:47
@g12i g12i requested a review from a team May 30, 2019 17:47
@g12i g12i added needs review A pull request ready to be reviewed and removed help wanted Extra attention is needed work in progress Author opened the pull-request but it's still being worked on labels May 30, 2019
@jakubdrozdek jakubdrozdek merged commit c2ab90e into master Jun 2, 2019
@jakubdrozdek jakubdrozdek deleted the hooks-reference branch June 2, 2019 19:35
@jakubdrozdek jakubdrozdek added 🌐 translated Translation is completed for this page and removed needs review A pull request ready to be reviewed labels Jun 2, 2019
@jakubdrozdek jakubdrozdek modified the milestones: Core Pages, Other Pages Jun 2, 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.

3 participants