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.

Sign up for GitHub

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

Jump to bottom

feat: viem client and sendEth #122

Closed

yawn-c111 wants to merge 3 commits into main from issue/118

Collaborator

yawn-c111 commented Sep 30, 2024 •

edited

Loading

関連イシュー

CLI: services/wallet.tsのgetWallet関数でProviderをいい感じにセットする #118

実装内容

clientを設定する関数を追加
sendEth関数を更新
sendEthのコマンドを追加

追加実装内容

client → wallet に変数名を変更
getWalletとgetAccountの2種類を用意
- getWalletでclientWalletを取得
sendEthの引数はaccountからwalletに変更
- それに伴い引数chainIdを別の関数に移動するなど

実装結果


          feat: viem client and sendEth

df13a8f

Collaborator Author

yawn-c111 commented Sep 30, 2024

コンフリクト直しておきます

yawn-c111 added 2 commits

September 30, 2024 21:44


          Merge branch 'main' into issue/118

ccefc2e


          update: getWallet and sendEth

015c0b9

yawn-c111 mentioned this pull request

Merged

Collaborator Author

yawn-c111 commented Oct 2, 2024

以下の別ブランチにマージの上、再度PRを送りましたので、こちらはクローズします！

Issue/104 #127

yawn-c111 closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet