PR #12659 Remove all `useEffect` usages

Remove all `useEffect` usages #12659

MichaReiser merged 5 commits into main from reduce-use-effect-usage

MichaReiser requested a review from

charliermarsh 1 year ago

MichaReiser added internal

charliermarsh commented on 2024-08-05

dhruvmanila commented on 2024-08-05

MichaReiser added playground

MichaReiser removed internal

MichaReiser commented on 2024-08-05

charliermarsh approved these changes on 2024-08-05

Move initialization out of editor

cc0830df

MichaReiser force pushed from ef99d783 to 7ff695ab 1 year ago

Remove `useEffect` from `useTheme` hook

6c0df94f

Remove `useEffect` usage for `checkResult`

e733433c

Fix top-level await build error

9c484a39

Show chrome during playground initialization

5f01bacf

MichaReiser force pushed from 7ff695ab to 5f01bacf 1 year ago

MichaReiser commented on 2024-08-05

MichaReiser requested a review from

charliermarsh 1 year ago

MichaReiser merged f5373352 into main 1 year ago

MichaReiser deleted the reduce-use-effect-usage branch 1 year ago

Reviewers

charliermarsh

dhruvmanila

Assignees

No one assigned

Labels

playground

Milestone

No milestone