-
-
Notifications
You must be signed in to change notification settings - Fork 353
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
#4581 gallery btn changes #4582
Conversation
✅ Deploy Preview for koda-nuxt ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@@ -39,7 +39,7 @@ watchEffect(async () => { | |||
const chainSymbol = urlPrefix.value === 'rmrk' ? unit.value : symbol | |||
|
|||
priceChain.value = `${parseFloat(price).toFixed(2)} ${chainSymbol}` | |||
priceUsd.value = `$${Math.round(Number(price) * Number(ksmPrice))}` |
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.
it's here to show "$" sign
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.
doesnt the issue mention to remove the dollar sign since we show usd already?
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.
Aight, then we don't need template literals here.
Should we keep "$" sign in GalleryItemOffers
too?
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.
Should we keep "$" sign in GalleryItemOffers too?
USD
good to go? |
Wait for me 😅 |
@exezbcz @yangwao I am not sure how to give that yet, found nothing here https://oruga.io/components/dropdown.html |
yes |
Code Climate has analyzed commit b03b5fd and detected 0 issues on this pull request. View more on Code Climate. |
pay 50 usd Some work was being done, yet we can move on Reopen if there are some things are not being still done |
😍 Perfect, I’ve sent the payout 🪅 Let’s grab another issue and get rewarded! |
Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type
Context
Before submitting pull request, please make sure:
Optional
Had issue bounty label?
Payout
Community participation
Screenshot 📸