Skip to content

build: begin migration to Typescript - #251

Closed
Tomahawk2191 wants to merge 2 commits into
isaac/update-sanity-configfrom
isaac/migrate-to-ts
Closed

build: begin migration to Typescript#251
Tomahawk2191 wants to merge 2 commits into
isaac/update-sanity-configfrom
isaac/migrate-to-ts

Conversation

@Tomahawk2191

@Tomahawk2191 Tomahawk2191 commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

What did I do?

Migrating the project from Javascript to Typescript.

How did I test it?

Describe in detail steps you used to test the changes you have made.

Required checks:

  • Did you conduct a self-review?
  • Have you written unit or integration tests?

What could go wrong in the future? What parts of your code should the reviewer pay the most attention to?

Describe aspects of the PR that may become problems in the future.

Additional Comments for the Reviewers

Screenshots

FIGMA

alt text

MY VERSION

alt text

Stack created with GitHub Stacks CLI • Give Feedback 💬

@fraander

fraander commented Sep 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

👷 Deploy Preview for sprightly-manatee-243873 processing.

Name Link
🔨 Latest commit 16efe73
🔍 Latest deploy log https://app.netlify.com/projects/sprightly-manatee-243873/deploys/6abca5f3d869fa00086f95a1

@Tomahawk2191
Tomahawk2191 changed the base branch from isaac/migrate-to-oxlint-oxfmt to isaac/migrate-to-vite September 5, 2026 20:11
@Tomahawk2191 Tomahawk2191 changed the title isaac/migrate to ts build: begin migration to Typescript Sep 6, 2026
@Tomahawk2191
Tomahawk2191 changed the base branch from isaac/migrate-to-vite to isaac/update-sanity-config September 6, 2026 18:44
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/update-sanity-config branch from 045f69a to 3fd7f6b Compare September 6, 2026 19:32
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/update-sanity-config branch from 3fd7f6b to e7f2212 Compare September 10, 2026 22:19
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/update-sanity-config branch from e7f2212 to e0857eb Compare September 11, 2026 01:10
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/update-sanity-config branch from e0857eb to 0f202b9 Compare September 11, 2026 01:11
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/migrate-to-ts branch 2 times, most recently from 549f1c2 to 96c2341 Compare September 11, 2026 01:14
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/update-sanity-config branch 2 times, most recently from 9bf6cd6 to cdf5173 Compare September 12, 2026 19:16
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/update-sanity-config branch from cdf5173 to 8dda183 Compare September 30, 2026 04:30
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/update-sanity-config branch from 8dda183 to d95f8dd Compare September 30, 2026 05:06
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/update-sanity-config branch from d95f8dd to a860f0e Compare September 30, 2026 05:57
@Tomahawk2191
fix: call useLocation in ScrollToTop
e70c0a4 
useLocation was referenced but never invoked, and the destructured key
was pathName rather than pathname. The effect dependency was therefore
always undefined and the scroll reset never fired on navigation.
@Tomahawk2191
fix: remove old pull_request_template.md
16efe73
@Tomahawk2191
Tomahawk2191 removed this pull request from stack #256 September 30, 2026 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

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

@Tomahawk2191 @fraander