Skip to content

New Indonesian Translation: Sharing State Between Components - #370

Merged
mhaidarhanif merged 43 commits into
reactjs:mainfrom
yaffalhakim1:sharing-state-between-components
May 1, 2023
Merged

New Indonesian Translation: Sharing State Between Components#370
mhaidarhanif merged 43 commits into
reactjs:mainfrom
yaffalhakim1:sharing-state-between-components

Conversation

@yaffalhakim1

@yaffalhakim1yaffalhakim1 commented Apr 28, 2023

Copy link
Copy Markdown
Contributor

Hi, this PR is for #330
New translation for page Sharing State Between Components

Closes#381

@yaffalhakim1yaffalhakim1 changed the title Sharing state between componentsNew Indonesian Translation: Sharing State Between ComponentsApr 28, 2023
@github-actions

Copy link
Copy Markdown

Size changes

Details

📦 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! 🙌

@github-actions

Copy link
Copy Markdown

Size changes

Details

📦 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! 🙌

@github-actions

Copy link
Copy Markdown

Size changes

Details

📦 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! 🙌

@github-actions

Copy link
Copy Markdown

Size changes

Details

📦 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! 🙌

@yaffalhakim1

Copy link
Copy Markdown
ContributorAuthor

Hi mas @resir014 and maintainers. This pr is ready to review. You can check in this commits:

@yaffalhakim1
yaffalhakim1 marked this pull request as ready for review April 29, 2023 04:01
resir014
resir014 previously requested changes Apr 29, 2023

@resir014resir014 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

First pass of code review. Here are some needed changes! @yaffalhakim1

Comment threadsrc/content/learn/sharing-state-between-components.md
Comment threadsrc/content/learn/sharing-state-between-components.md Outdated
Comment threadsrc/content/learn/sharing-state-between-components.md Outdated
Comment threadsrc/content/learn/sharing-state-between-components.md Outdated
Comment threadsrc/content/learn/sharing-state-between-components.md Outdated
Comment threadsrc/content/learn/sharing-state-between-components.md Outdated
@yaffalhakim1

Copy link
Copy Markdown
ContributorAuthor

First pass of code review. Here are some needed changes! @yaffalhakim1

alright mas, I will commit the suggestion ASAP while waiting for other maintainers to give their reviews.

yaffalhakim1and others added 4 commits April 29, 2023 13:00
Co-authored-by: Resi Respati <resir014@gmail.com>
Co-authored-by: Resi Respati <resir014@gmail.com>
Co-authored-by: Resi Respati <resir014@gmail.com>
Co-authored-by: Resi Respati <resir014@gmail.com>
yaffalhakim1and others added 3 commits May 1, 2023 18:18
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
yaffalhakim1and others added 2 commits May 1, 2023 18:54
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
applying all suggestion from reviewers
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
@yaffalhakim1

Copy link
Copy Markdown
ContributorAuthor

Thanks for contributing! I have a few more suggestions here. 😁

surely, I missed a lot 😓, thank you for the review mas. ready to re-review

@zainfathonizainfathoni left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 💯
Approved with a minor suggestion for paraphrasing. 😁

Thanks for contributing! 🙏

Comment threadsrc/content/learn/sharing-state-between-components.md Outdated
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
@mhaidarhanif
mhaidarhanif dismissed resir014’s stale reviewMay 1, 2023 23:41

there are already 3 review approvals

@mhaidarhanif
mhaidarhanif merged commit 0d347ec into reactjs:mainMay 1, 2023
r17x pushed a commit that referenced this pull request May 8, 2023
Co-authored-by: Resi Respati <resir014@gmail.com>
Co-authored-by: M Haidar Hanif <git@mhaidarhanif.com>
Co-authored-by: Zain Fathoni <me@zainf.dev>
Co-authored-by: Zain Fathoni <zain.fathoni@gmail.com>
Sign up for freeto 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.

Sharing State Between Components

5 participants

@yaffalhakim1@mhaidarhanif@resir014@zainfathoni@r17x