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

[Spike] Update evm/use-feeProxy example #35

Merged
merged 9 commits into from
Oct 3, 2023

Conversation

ken-futureverse
Copy link
Contributor

@ken-futureverse ken-futureverse commented Sep 20, 2023

Summary

Update the evm/use-feeProxy example to be in the same format as the new substrate/use-feeProxy.

I also incorporate the change you have in this #34 and add in substrate/use-feeProxy/src/callEVMCall.ts. The estimate is a bit tricky, but I think I got it to be relatively accurate.

Other changes:

  • Move common asset ids to a single module
  • Update the output to use pino-pretty
  • Switch all use-feeProxy examples to use tsx for better performance, will go through and update the rest in a separate PR

Checklist

  • Add description

Sorry, something went wrong.

@ken-futureverse ken-futureverse marked this pull request as ready for review September 21, 2023 02:05
@ken-futureverse ken-futureverse enabled auto-merge (squash) September 21, 2023 02:06
Copy link
Contributor

@KarishmaBothara KarishmaBothara left a comment

Choose a reason for hiding this comment

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

lgtm

@ken-futureverse ken-futureverse merged commit 17fb2e7 into main Oct 3, 2023
1 check passed
@ken-futureverse ken-futureverse deleted the spike/update-feeProxy-evm branch October 3, 2023 19:51
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.

None yet

2 participants