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: fix nft demo gas estimation #42

Merged
merged 3 commits into from
Dec 12, 2024

Conversation

ly0va
Copy link
Member

@ly0va ly0va commented Dec 12, 2024

No description provided.

Copy link

github-actions bot commented Dec 12, 2024

Visit the preview URL for this PR (updated for commit ebaf683):

https://zksync-auth-server-staging--pr42-lyova-fix-nft-quest-492nrn6y.web.app

(expires Thu, 19 Dec 2024 17:06:15 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 509a9c9ea42583076f531c53cf2979c544d5d0b7

MexicanAce
MexicanAce previously approved these changes Dec 12, 2024
Copy link
Contributor

@MexicanAce MexicanAce left a comment

Choose a reason for hiding this comment

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

LGTM. FYI, using 0n for feeLimit will default to a value of something like 0.001 ETH (to be fixed in the future)

@MexicanAce MexicanAce merged commit 68bdf0d into main Dec 12, 2024
7 checks passed
@MexicanAce MexicanAce deleted the lyova-fix-nft-quest-gas-estimation branch December 12, 2024 17:10
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