Repository navigation
[Bug] Fix missing react-query dependency and configure QueryClientProvider #63
Description
Background
In sharif-chain-scope/src/hooks/useTransactionHistory.ts, the hook imports and uses useQuery from react-query. However, react-query is not listed in the dependencies of package.json. Additionally, the React application has no root-level QueryClient initialized or wrapped with QueryClientProvider, which will cause runtime crashes.
Description
Add the missing dependency to package.json and properly wrap the React app layout in the query provider.
Acceptance Criteria
- Add
@tanstack/react-queryto dependencies inpackage.json. - Instantiate a
QueryClientat the application root (e.g.,src/index.tsxorsrc/App.tsx). - Wrap the
Dashboardor root component within<QueryClientProvider client={queryClient}>. - Verify that running
npm run devbuilds successfully and doesn't crash in the browser.
Activity
WilliamK112 commentedon Aug 1, 2026
Implemented in #86.
The PR replaces the legacy React Query dependency, migrates the hook to the v5 API, adds the missing Vite entry point, and mounts
DashboardinsideQueryClientProvider.Validation completed:
npm cinpm test -- --runInBand— 79 tests passednpm run build— passedThe current red
PR Automation / pr-reviewcheck is unrelated to the patch: the repository workflow fails while trying to post its welcome comment because itsGITHUB_TOKENonly has read permissions (403 Resource not accessible by integration).ranjeet150 commentedon Sep 15, 2026
Hey maintainers,
I would love to take this on. My planned approach:
[Bug] Fix missing react-query dependency and configure QueryClientProviderfollowing the existing UI design system, accessibility standards, and responsive styling.Excited to work on this—please assign to me and I will submit a clean PR shortly!