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

Fix link in thinking-in-react.md #1115

Merged
merged 2 commits into from
Jan 4, 2025
Merged

Conversation

rammba
Copy link
Contributor

@rammba rammba commented Jan 4, 2025

Hello @lumirlumir @devwqc,
I'm really happy to see that you're maintaining this translation. As a fellow Serbian translator I must admit it's a really hard and extensive job.

I've created this PR based on the issue (reactjs/react.dev#7398) on main repo which I noticed.
Since I don't speak korean, I hope you're ok with me contributing to this repo 😉
I wish you all luck with maintaining and improving this repo 🚀

Copy link

vercel bot commented Jan 4, 2025

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
ko-legacy-reactjs-org ⬜️ Ignored (Inspect) Visit Preview Jan 4, 2025 4:00pm

Copy link

github-actions bot commented Jan 4, 2025

Size changes

📦 Next.js Bundle Analysis for undefined

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

This PR introduced no changes to the JavaScript bundle! 🙌

@lumirlumir
Copy link
Collaborator

lumirlumir commented Jan 4, 2025

@rammba Thank you for your hard work on React! I’m truly impressed by your passion.

Yes, maintaining a translation repository can be challenging, but it’s incredibly meaningful. 😄


This PR looks good to me. However, I’ll need to merge #1114 first before merging this commit.

I’ve also left a comment on the i18n issue I created! (reactjs/react.dev#7399)

@lumirlumir lumirlumir self-assigned this Jan 4, 2025
@lumirlumir lumirlumir added the DO NOT CLOSE OR MERGE 닫으면 안되는 이슈 또는 PR label Jan 4, 2025
@lumirlumir lumirlumir marked this pull request as draft January 4, 2025 05:20
@rammba
Copy link
Contributor Author

rammba commented Jan 4, 2025

Thanks for the kind words! Feel free to take your time with merging major changes first.

@lumirlumir lumirlumir marked this pull request as ready for review January 4, 2025 16:00
@lumirlumir lumirlumir merged commit 3825c05 into reactjs:main Jan 4, 2025
6 checks passed
@rammba rammba deleted the fix/usestate-link branch January 5, 2025 01:28
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.

2 participants