Added useFlowQueryRaw hook #2523
Conversation
🦋 Changeset detectedLatest commit: 43cb3d8 The changes in this PR will be included in the next version bump. This PR includes changesets to release 8 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new hook, useFlowQueryRaw, to execute a Cadence script and obtain its raw query response without decoding it. Key changes include:
- Implementation of the useFlowQueryRaw hook integrating with react-query and fcl.rawQuery.
- Comprehensive tests for various scenarios including error handling, argument updates, and disabling queries.
- Updates and re-exports across related files to support the new hook.
Reviewed Changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/kit/src/hooks/useFlowQueryRaw.ts | New hook implementation for executing raw queries |
| packages/kit/src/hooks/useFlowQueryRaw.test.ts | Added tests covering multiple use cases for the new hook |
| packages/kit/src/hooks/index.ts | Exports new hook |
| packages/kit/src/mocks/fcl.ts | Added mock for rawQuery |
| packages/fcl/src/fcl.ts | Updated export for rawQuery |
| packages/fcl-core/src/fcl-core.ts | Updated export for rawQuery |
| packages/fcl-core/src/exec/raw-query.ts | New implementation of rawQuery function |
| packages/fcl-core/src/exec/query.ts | Updated query function to work with rawQuery |
| packages/fcl-core/src/exec/query.js | Removal of outdated query implementation |
| .changeset/wicked-bikes-peel.md | Changeset documenting the new hook |
|
|
||
| // Encode the arguments to a JSON-CDC object so they can be deterministically | ||
| // serialized and used as the query key. | ||
| const encodedArgs = args?.(fcl.arg, fcl.t)?.map((x: any) => |
There was a problem hiding this comment.
This is in the useFlowQuery file as well. Can we move it somewhere its shared?
There was a problem hiding this comment.
Yep, created encodeQueryArgs function and shared between hooks!
Yes, tested! |
Close #2407