translate you-might-not-need-an-Effect by enesakkose · Pull Request #469 · reactjs/tr.react.dev
Navigation Menu
{{ message }}
reactjs / tr.react.dev Public
- Notifications You must be signed in to change notification settings
- Fork 171
Merged
alioguzhan merged 5 commits intoreactjs:mainfrom
Jun 18, 2023Merged
translate you-might-not-need-an-Effect#469
alioguzhan merged 5 commits intoreactjs:mainfrom
translate you-might-not-need-an-Effect#469
alioguzhan merged 5 commits intoreactjs:mainfrom
Conversation
Copy link
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! 🙌
enesakkose added 2 commits
June 15, 2023 22:04…ub.com/enesakkose/tr.react.dev into translate-You-Might-Not-Need-an-Effect
alioguzhan
self-requested a review
alioguzhan requested changes Jun 17, 2023
Copy link
Member
alioguzhan
left a comment
alioguzhan
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tesekkurler. Elinize saglik. Genel olarak guzel bir ceviri olmus. Degisiklik yapilmasi gereken yerleri belirttim. Sonrasinda merge edebiliriz.
src/content/learn/you-might-not-need-an-effect.md
Outdated
Show resolved
Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md
Outdated
Show resolved
Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md
Outdated
Show resolved
Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md
Outdated
Show resolved
Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md
Outdated
Show resolved
Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md
Outdated
Show resolved
Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md
Outdated
Show resolved
Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md
Outdated
Show resolved
Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md
Outdated
Show resolved
Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md
Outdated
Show resolved
Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
enesakkose
requested a review
from alioguzhan
alioguzhan approved these changes Jun 18, 2023
Copy link
Member
alioguzhan
left a comment
alioguzhan
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tesekkurler. Merge ediyorum.
alioguzhan
merged commit
2ed5c55
into
reactjs:main
alioguzhan
mentioned this pull request
Copy link
Contributor Author
enesakkose
commented
Jun 18, 2023
enesakkose commented
Jun 18, 2023Rica ederim.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment