Skip to content

Conversation

sannek
Copy link
Contributor

@sannek sannek commented Sep 25, 2025

Reverts #8799

There's a type error that I don't understand, so it won't deploy anyway. Making me think maybe we do use these sandboxes? I don't have time to properly look into it.

src/app/pages/Dashboard/Content/routes/Recent/index.tsx(92,30): error TS2339: Property 'sandboxes' does not exist on type 'SidebarState'. <- this is the type error, but I don't understand because when I deployed to stream it did just work and showed me recent sandboxes. Here we're trying to read the sandboxes from the sidebar state, but I'm pretty sure we have them elsewhere ij ust don't know where.

(The linting errors were related to unused variables, but those are easily fixed)

Copy link

codesandbox bot commented Sep 25, 2025

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@sannek sannek merged commit aa99921 into main Sep 25, 2025
9 of 14 checks passed
@sannek sannek deleted the revert-8799-sanne/perf-remove-unused-gql-fields branch September 25, 2025 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants