Skip to content

Translate useEffect - #897

Merged
titovmx merged 17 commits into
reactjs:mainfrom
mekarthedev:translate-useeffect
May 1, 2025
Merged

Translate useEffect#897
titovmx merged 17 commits into
reactjs:mainfrom
mekarthedev:translate-useeffect

Conversation

@mekarthedev

@mekarthedevmekarthedev commented Jun 4, 2023

Copy link
Copy Markdown
Contributor

@mekarthedevmekarthedev changed the title Translate useeffectTranslate useEffectJun 4, 2023
@github-actions

github-actionsBot commented Jun 4, 2023•
edited
Loading

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

@mekarthedevmekarthedev mentioned this pull request Jun 4, 2023
Open
titovmx
titovmx approved these changes Aug 15, 2023
View reviewed changes

@titovmxtitovmx 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.

пара небольших замечаний, в остальном всё классно, спасибо за перевод!

Comment threadsrc/content/reference/react/useEffect.md Outdated
Comment threadsrc/content/reference/react/useEffect.md Outdated
@mekarthedev@titovmx
Apply suggestions from code review
3f3e8a9
Co-authored-by: Maxim <mtitov92@gmail.com>
AbdullahWins
AbdullahWins approved these changes Aug 22, 2023
View reviewed changes

@AbdullahWinsAbdullahWins left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

well done!

@titovmx

titovmx commented May 1, 2025

Copy link
Copy Markdown
Collaborator

@mekarthedev добрый день, я не могу смержить ПР из-за конфликтов и не могу поправить их в вашей ветке. Если у вас будет возможность вернуться к этому ПРу, буду очень благодарен и после сразу смогу смёржить. Видимо в 2023 я понадеялся, что ещё придёт второй ревьюер и оставил этот ПР висеть.

@vercel

vercelBot commented May 1, 2025•
edited
Loading

Copy link
Copy Markdown

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

1 Skipped Deployment
NameStatusPreviewCommentsUpdated (UTC)
ru-legacy-reactjs-org⬜️ Ignored (Inspect)Visit PreviewMay 1, 2025 3:53pm

@mekarthedev

mekarthedev commented May 1, 2025

Copy link
Copy Markdown
ContributorAuthor

@titovmx готово.

@titovmx
titovmx merged commit 8b58e2d into reactjs:mainMay 1, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@titovmxtitovmxtitovmx approved these changes
+1 more reviewer
@AbdullahWinsAbdullahWinsAbdullahWins approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mekarthedev@titovmx@AbdullahWins