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

Fix burner connector raw transaction #16

Merged
merged 2 commits into from
May 15, 2024

Conversation

technophile-04
Copy link
Collaborator

Description :

Sending transaction with useSendTranscation were failing, because we were passing hex gas values to viems's sendTransaction write actions instead it accepts bigInt

Also did some digging why useSendTransaction was not working and useTransactor was working was because when you use sendTransaction from wagmi hook it passes some gas value, and when you do walletClient.sendTransaction (we are doing this in useWalletClient) this sends gas value as 0

Screenshot 2024-05-15 at 5 30 44 PM

Solution, found this file in viem reversed engineered it and passed all the options accepted by viem's sendTransaction.

Didn't handled advanced options like kzg and blobs we can maybe find better / other way in future PR's

@technophile-04
Copy link
Collaborator Author

technophile-04 commented May 15, 2024

Create this branch to test : https://github.com/scaffold-eth/burner-connector/tree/test-fix-burner-raw

Screen.Recording.2024-05-15.at.7.01.04.PM.mov

It redecued 2 ETH because I am sending 1 eth transaction with both methods

Also tested nicely, by making transaction

@technophile-04 technophile-04 merged commit f917ed6 into main May 15, 2024
1 check passed
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.

1 participant