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: translate API Reference -> react -> useInsertionEffect #516

Merged
merged 12 commits into from Jul 3, 2023

Conversation

igihcksn
Copy link
Contributor

@igihcksn igihcksn commented Apr 30, 2023

Closes #404

Description

Translate the useInsertionEffect page.
Page URL: https://id.react.dev/reference/react/useInsertionEffect

@mazipan
Copy link
Collaborator

mazipan commented May 10, 2023

Can you help to fix the conflict first @igihcksn?

@mazipan
Copy link
Collaborator

mazipan commented May 18, 2023

@resir014 can you help to fix the conflict first?

@igihcksn
Copy link
Contributor Author

Can you help to fix the conflict first @igihcksn?

Sorry for late reply mas, i've resolve the conflict and update the translation, kindly need your help to review

src/content/reference/react/useInsertionEffect.md Outdated Show resolved Hide resolved
src/content/reference/react/useInsertionEffect.md Outdated Show resolved Hide resolved
src/content/reference/react/useInsertionEffect.md Outdated Show resolved Hide resolved
src/content/reference/react/useInsertionEffect.md Outdated Show resolved Hide resolved
src/content/reference/react/useInsertionEffect.md Outdated Show resolved Hide resolved
src/content/reference/react/useInsertionEffect.md Outdated Show resolved Hide resolved
src/content/reference/react/useInsertionEffect.md Outdated Show resolved Hide resolved
src/content/reference/react/useInsertionEffect.md Outdated Show resolved Hide resolved
@igihcksn
Copy link
Contributor Author

Thanks for the feedback mas @mazipan, i've update base on your sugesstions, kindly need help to check

Co-authored-by: Irfan Maulana <mazipanneh@gmail.com>
Copy link
Member

@resir014 resir014 left a comment

Choose a reason for hiding this comment

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

Some requested changes. @igihcksn

src/content/reference/react/useInsertionEffect.md Outdated Show resolved Hide resolved
src/content/reference/react/useInsertionEffect.md Outdated Show resolved Hide resolved
src/content/reference/react/useInsertionEffect.md Outdated Show resolved Hide resolved
igihcksn and others added 3 commits May 26, 2023 12:49
Co-authored-by: Resi Respati <resir014@gmail.com>
Co-authored-by: Resi Respati <resir014@gmail.com>
Co-authored-by: Resi Respati <resir014@gmail.com>
@igihcksn
Copy link
Contributor Author

Some requested changes. @igihcksn

All done mas, kindly need your help to check

@mazipan
Copy link
Collaborator

mazipan commented May 30, 2023

Help recheck @resir014

@mazipan
Copy link
Collaborator

mazipan commented Jun 22, 2023

@resir014

@github-actions
Copy link

github-actions bot commented Jun 22, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@resir014 resir014 changed the title Translation of useInsertionEffect docs: translate API Reference -> react -> useInsertionEffect Jul 3, 2023
@resir014 resir014 merged commit c3d7957 into reactjs:main Jul 3, 2023
4 checks passed
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.

useInsertionEffect
4 participants