Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: useEstimateFees hook #422

Merged
merged 3 commits into from
Apr 3, 2024

Conversation

ugur-eren
Copy link
Contributor

Context

As described in the issue #411, dapps sometimes needs fees estimation before doing the actual contract call. For this, I've created the useEstimateFees hook.

Changes in this Pull Request

  • Added useEstimateFees custom hook.
  • Added relevant documentation in the docs with usage example.

@ugur-eren ugur-eren requested a review from fracek as a code owner March 30, 2024 19:47
Copy link

netlify bot commented Mar 30, 2024

Deploy Preview for starknet-react ready!

Name Link
🔨 Latest commit 78b124d
🔍 Latest deploy log https://app.netlify.com/sites/starknet-react/deploys/660ab0fdf3fcb6000829dd3d
😎 Deploy Preview https://deploy-preview-422--starknet-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@fracek
Copy link
Contributor

fracek commented Apr 1, 2024

Looks good! Can you add a changelog with pnpm changeset (minor version bump) so then I can make a release?

@ugur-eren
Copy link
Contributor Author

Hi @fracek , I added the changelog 2 days ago but seems like forgot to mention it, sorry for late reply.

@fracek
Copy link
Contributor

fracek commented Apr 3, 2024

Thank you!

@fracek fracek merged commit cca5816 into apibara:main Apr 3, 2024
5 checks passed
@github-actions github-actions bot mentioned this pull request Apr 3, 2024
fracek added a commit that referenced this pull request Apr 3, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @starknet-react/core@2.6.0

### Minor Changes

- [#422](#422)
[`78b124d`](78b124d)
Thanks [@ugur-eren](https://github.com/ugur-eren)! - Add useEstimateFees
hook to estimate smart contract calls fees
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