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.

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

Update web3js #89

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

Update web3js #89

wants to merge 2 commits into from

Conversation

MkDev11
Copy link

@MkDev11 MkDev11 commented Apr 20, 2023

update web3js to accommodate config tuple

@vercel
Copy link

vercel bot commented Apr 20, 2023

@MkDev11 is attempting to deploy a commit to the Artgene Team on Vercel.

A member of the Team first needs to authorize it.

@what-the-diff
Copy link

what-the-diff bot commented Apr 20, 2023

PR Summary

  • Added a new function to obtain mint price from contract's configuration
    A new function getMintPriceFromConfig() has been introduced to fetch the mint price from the contract's configuration file.
  • Updated the logic of getMintPrice() method
    The updated logic now handles cases when there is no custom mint price. It will look for a constant value in the DEFAULTS object or try to use any available method ('price', 'cost', 'public_sale_price', 'getPrice', or 'salePrice') to determine the mint price.

@vercel
Copy link

vercel bot commented Apr 20, 2023 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
nftcomponents ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 20, 2023 0:44am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@MkDev11