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

Docs/hooks api reference #272

Merged
merged 39 commits into from
Jan 17, 2023
Merged

Conversation

PReichetanz
Copy link
Contributor

Translation for Hooks API Reference

All comments and ideas for further improving the translation are very much appreciated!

content/docs/hooks-reference.md Outdated Show resolved Hide resolved
content/docs/hooks-reference.md Outdated Show resolved Hide resolved
content/docs/hooks-reference.md Outdated Show resolved Hide resolved
content/docs/hooks-reference.md Outdated Show resolved Hide resolved
content/docs/hooks-reference.md Outdated Show resolved Hide resolved
content/docs/hooks-reference.md Outdated Show resolved Hide resolved
content/docs/hooks-reference.md Outdated Show resolved Hide resolved
content/docs/hooks-reference.md Outdated Show resolved Hide resolved
content/docs/hooks-reference.md Outdated Show resolved Hide resolved
content/docs/hooks-reference.md Outdated Show resolved Hide resolved
@ph1p ph1p added needs review from author First review completed. All changes are processed and a final review is required. and removed completed - needs review 🧐 labels Jan 3, 2023
PReichetanz and others added 18 commits January 5, 2023 08:42
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
PReichetanz and others added 4 commits January 5, 2023 08:54
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
PReichetanz and others added 5 commits January 5, 2023 09:14
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
Co-authored-by: Phil <phil@capshake.com>
@vercel
Copy link

vercel bot commented Jan 5, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
de.reactjs.org 🔄 Building (Inspect) Jan 5, 2023 at 8:17AM (UTC)

Co-authored-by: Phil <phil@capshake.com>
@PReichetanz
Copy link
Contributor Author

Hey @ph1p,
thank you very much for your review and suggestions, I have learned a lot!

I have agreed on almost all issues you have remarked; there are, however, three issues left where I am not sure how to handle them.

Could you please have a look again so that we can discuss what do to here?

Thank you very much again!

@github-actions
Copy link

github-actions bot commented Jan 5, 2023

Size Changes

📦 Next.js Bundle Analysis

This analysis was generated by the next.js bundle analysis action 🤖

This PR introduced no changes to the javascript bundle 🙌

Co-authored-by: Paul Reichetanz <33567019+PReichetanz@users.noreply.github.com>
@ph1p
Copy link
Member

ph1p commented Jan 5, 2023

Thank you for the quick response (: I commented a little but after that it should be ready to merge

ph1p and others added 2 commits January 17, 2023 09:37
Co-authored-by: Paul Reichetanz <33567019+PReichetanz@users.noreply.github.com>
@ph1p ph1p removed the needs review from author First review completed. All changes are processed and a final review is required. label Jan 17, 2023
@ph1p
Copy link
Member

ph1p commented Jan 17, 2023

Sorry for the delay. All problems are solved now and we are ready to merge. Thank you! (:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants