Skip to content

forwardRef.md#57

Merged
bielarusajed merged 4 commits intomainfrom
forwardRef
Jan 11, 2024
Merged

forwardRef.md#57
bielarusajed merged 4 commits intomainfrom
forwardRef

Conversation

@bielarusajed
Copy link
Collaborator

@bielarusajed bielarusajed commented Dec 31, 2023

No description provided.

@bielarusajed
Copy link
Collaborator Author

bielarusajed commented Dec 31, 2023

На абмеркаванне: Imperative Handle пераклаў як загаднае кіраванне. Як варыянт, можна яшчэ пакінуць імператыўнае кіраванне.

@github-actions
Copy link

github-actions bot commented Dec 31, 2023 •
edited
Loading

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

@bielarusajed
Copy link
Collaborator Author

bielarusajed commented Dec 31, 2023

А, і яшчэ. Перад тым, як мержыць, думаю, варта дачакацца адказу па гэтым issue: reactjs/react.dev#6525

То, можна пакуль правесці рэв’ю і адкласці PR

@bielarusajed bielarusajed mentioned this pull request Dec 31, 2023
Open
zubialevich
zubialevich reviewed Jan 8, 2024
View reviewed changes
@bielarusajed
Copy link
Collaborator Author

bielarusajed commented Jan 11, 2024

Так, адкрытымі на абмеркаванне застаюцца пытанні датычныя render/rendering і пра нячыстыя функцыі.

Таксама запушыў фікс гэтага артыкула з асноўнага рэпазіторыя, які да нас яшчэ не даляцеў, як я бачу. Потым, як прыйдзе PR, пэўна, вазьму яго на сябе.

@zubialevich
Copy link
Collaborator

zubialevich commented Jan 11, 2024

Так, адкрытымі на абмеркаванне застаюцца пытанні датычныя render/rendering і пра нячыстыя функцыі.

Таксама запушыў фікс гэтага артыкула з асноўнага рэпазіторыя, які да нас яшчэ не даляцеў, як я бачу. Потым, як прыйдзе PR, пэўна, вазьму яго на сябе.

Наконт render/rendering - пакідаем як ёсць. Гэта я нешта не тое ляпнуў)

Не маю нічога супраць "нячыстых функцый". Проста слова "нячыстасці" неяк вуха разанула. Там можна перафармуляваць сказ крыху проста

@bielarusajed
Copy link
Collaborator Author

bielarusajed commented Jan 11, 2024

Ну то тады паправіў

@bielarusajed bielarusajed merged commit 1cd5c85 into main Jan 11, 2024
@bielarusajed bielarusajed deleted the forwardRef branch January 11, 2024 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zubialevich zubialevich zubialevich left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bielarusajed @zubialevich