Skip to content

Conversation

@allison-truhlar
Copy link
Collaborator

@allison-truhlar allison-truhlar commented Oct 30, 2025

This pull request introduces several improvements and refactors to the frontend codebase, focusing on migrating data fetching and state management to React/Tanstack Query.

React Query Integration:

  • Added @tanstack/react-query, @tanstack/react-query-devtools, and @tanstack/eslint-plugin-query dependencies.
  • Replaced usage of useEffects for data fetching with the queries.
  • Updated frontend unit test utilities to wrap components in a QueryClientProvider.

Miscellaneous:

  • Fixed Playwright tests by forcing them to run in parallel, have each test create a unique folder to generate test files in, and clearing the shared database at the end of each test.
  • Added Playwright test report directory to .prettierignore to avoid formatting issues with generated reports.
  • Fixed error on push: 'Can't find lefthook in PATH' by explicitly setting the lefthook location and command in lefthook.yml. This error resulted from the refactor that moved the frontend code, where lefthook is installed via npm.
  • Added a frontend directory CLAUDE.md file to aid Claude Code in following the development patterns in this directory across collaborators.

Note: this PR now has several other PRs merged into it:
#236
#237
#238
#239

- includes refactoring components that used FileBrowserContext to use these new queries where applicable
…t-permissions

fix: viewing folder without permissions shows error message
…mp/fileglancer into initial-tanstack-query-test
- this was an attempt to eliminate what appeared to be duplicate calls to the generate Neuroglancer state functions
- also show zarr logo as placeholder if there is an error
fix: show loading indicator for zarr metadata
@allison-truhlar allison-truhlar merged commit b1f34ea into main Nov 6, 2025
7 checks passed
@allison-truhlar allison-truhlar deleted the initial-tanstack-query-test branch November 6, 2025 21:23
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.

4 participants