-
Notifications
You must be signed in to change notification settings - Fork 3
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: name activation price label in dashboard #356
Conversation
src/utils/adapters.js
Outdated
? formatAettosToAe(chainName.last_bid.tx.name_fee) | ||
: formatAettosToAe(chainName.info.claims[chainName.info.claims.length - 1].tx.name_fee), | ||
isAuction: isAuction(chainName.name), | ||
price: formatAettosToAe(chainName.info.claims[chainName.info.claims.length - 1].tx.name_fee), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apparently there was some change in the mdw since we implemented this code and this code can be used to get the price for both auctioned name and a direct buy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you tried this? Its a new addition to JS https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/at
Deployed to https://pr-356-aescan.stg.aepps.com |
@lukeromanowicz I think "Auction" is sufficient as label |
@marc0olo roger that, adjusted 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice change, I like 👍
Please check my suggestion
src/utils/adapters.js
Outdated
? formatAettosToAe(chainName.last_bid.tx.name_fee) | ||
: formatAettosToAe(chainName.info.claims[chainName.info.claims.length - 1].tx.name_fee), | ||
isAuction: isAuction(chainName.name), | ||
price: formatAettosToAe(chainName.info.claims[chainName.info.claims.length - 1].tx.name_fee), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you tried this? Its a new addition to JS https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/at
@janmichek love it, thank you! I haven't seen that before |
Cool, yea its a new thing |
Are PR preview links broken? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job 🚀
Description
Resolves #328
Demo
Checklist: