Skip to content

Typescript Migration: client/modules/IDE/hook and client/modules/IDE/utils #3787

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.

Sign up for GitHub

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

Jump to bottom
Open
clairep94 wants to merge 23 commits into
base: develop
Choose a base branch
from
Open

Typescript Migration: client/modules/IDE/hook and client/modules/IDE/utils #3787

clairep94 wants to merge 23 commits into from
+268 −229

Conversation

@clairep94
Copy link
Collaborator

@clairep94 clairep94 commented Jan 10, 2026

Migrate client/modules/IDE/hooks and client/modules/IDE/utils

Changes:

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • has no typecheck errors (npm run typecheck)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123
  • meets the standards outlined in the accessibility guidelines

@clairep94
consoleStyles: update to ts, no verify
5ee0bef
@clairep94
add .svg?byContent and .svg?byUrl to custom types
2c3f3d9
@clairep94
consoleStyles: resolve type errors & update to named export
7445aec
@clairep94
hooks/index: update to ts
2413816
@clairep94
hooks/useWhatPage: update to ts, no-verify
3dbcf68
@clairep94
hooks/useWhatPage: update to named export & resolve type errors
f6fb4c4
@clairep94
hooks/useIsMobile: update to ts, no-verify
d6538ca
@clairep94
hooks/useIsMobile: update to named export & resolve type errors
66ddcb7
@clairep94
hooks/custom-hooks: update to ts, no-verify
164ce98
@clairep94
hooks/custom-hooks: delete unused hooks, resolve type errors
f8ceb97
@clairep94
hooks/custom-hooks: update file to useDidUpdate
b897501
@clairep94
IDE/hooks/useInterval: update to ts, no-verify
0f40933
@clairep94
IDE/hooks/useInterval: update to named export and resolve type errors
17a18c2
@clairep94
hooks/useIsMobile: add types for react-responsive
515f2ca
@clairep94
hooks/useDidUpdate: minor cleanup
e22ae36
@clairep94
hooks/useP5Version: update to ts, no-verify
218ad0f
@clairep94
hooks/useP5Versions: resolve type errors
3164d15
@clairep94
hooks/useSketchActions: update to ts, no-verify
814d9d7
@clairep94
hooks/useSketchActions: update to named export, resolve type errors
3c5a071
@clairep94
hooks/useSketchActions: memoise methods
f2e0d88
@clairep94
hooks/useHandleMessageEvent: update to ts, no-verify
ad4cdc0
@clairep94
hooks/useHandleMessageEvent: resolve type errors, add Message type fr…
04e839a 
…om console-feed
clairep94
clairep94 commented Jan 10, 2026
View reviewed changes
client/modules/IDE/hooks/useSketchActions.ts
Copy link
Collaborator Author

@clairep94 clairep94 Jan 10, 2026

Choose a reason for hiding this comment

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

memoized the methods in useSketchActions

@clairep94 clairep94 changed the title Client modules ide p1 Typescript Migration: client/modules/IDE/hook and client/modules/IDE/utils Jan 10, 2026
@clairep94 clairep94 requested review from khanniie and raclim January 10, 2026 21:25
@clairep94 clairep94 added the Area: Typescript Migration Related to Typescript Migration after the initial pr05 grant 2025 migration work label Jan 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@raclim raclim Awaiting requested review from raclim

@khanniie khanniie Awaiting requested review from khanniie

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Area: Typescript Migration Related to Typescript Migration after the initial pr05 grant 2025 migration work

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@clairep94