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: bridge hooks #1950

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

feat: bridge hooks #1950

wants to merge 7 commits into from

Conversation

0xAlec
Copy link
Contributor

@0xAlec 0xAlec commented Feb 11, 2025

What changed? Why?
scaffold hooks for bridge UI component and misc. utils and constants

  • useAppchainConfig
    retrieves the onchain configuration for an appchain (id and contracts)
  • useDeposit
    wraps wagmi hooks for depositing funds to the appchain
  • useBalance
    retrieves current user balance (ETH or ERC-20)

(withdrawals will be added in a later PR)

Notes to reviewers

How has it been tested?
unit tests

Copy link

vercel bot commented Feb 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
onchainkit-coverage ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 11, 2025 8:11pm
onchainkit-playground ❌ Failed (Inspect) Feb 11, 2025 8:11pm
onchainkit-routes ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 11, 2025 8:11pm

@0xAlec 0xAlec changed the title Alec/bridge hooks bridge hooks Feb 11, 2025
@0xAlec 0xAlec marked this pull request as ready for review February 11, 2025 03:43
@0xAlec 0xAlec changed the title bridge hooks feat: bridge hooks Feb 11, 2025
@github-actions github-actions bot added the root label Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

1 participant