Skip to content

Add query options estimating fees#12

Merged
gndelia merged 4 commits intomasterfrom
add-query-options-estimating-fees
Mar 20, 2026
Merged

Add query options estimating fees#12
gndelia merged 4 commits intomasterfrom
add-query-options-estimating-fees

Conversation

@gndelia
Copy link
Contributor

@gndelia gndelia commented Mar 20, 2026

Description

This PR adds queryOptions to the useEstimateFee hook so I can estimate fees in sync code

Checklist

  • Manual testing passed.
  • Automated tests added, or N/A.
  • Documentation updated, or N/A.
  • Environment variables set in CI, or N/A.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refactors useEstimateFees to expose a reusable React Query options builder, enabling fee estimation via queryClient.fetchQuery/ensureQueryData outside of React hook usage.

Changes:

  • Added estimateFeesQueryKey and exported estimateFeesQueryOptions for fee estimation.
  • Refactored useEstimateFees to use useQuery with the new query options (instead of wagmi hooks directly).
  • Bumped package version to 1.5.0.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/useEstimateFees.ts Adds exported React Query options for estimating fees and refactors the hook to use them.
package.json Version bump to reflect the new exported API surface.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gndelia gndelia enabled auto-merge March 20, 2026 17:02
@gndelia gndelia merged commit c462eca into master Mar 20, 2026
11 checks passed
@gndelia gndelia deleted the add-query-options-estimating-fees branch March 20, 2026 18:19
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