-
-
Notifications
You must be signed in to change notification settings - Fork 347
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: 5003 nft card without name #5013
fix: 5003 nft card without name #5013
Conversation
✅ Deploy Preview for koda-nuxt ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
add more fallback maybe?
-
on explore pages https://deploy-preview-5013--koda-nuxt.netlify.app/rmrk/explore/items?page=1
-
identity components
- user/collection pages
@preschian still sometimes getting this btw: |
added |
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.
otherwise code lgtm
yes, the PR I added before only works on github actions. on local, I think we can just restart the command or trigger save file to reload |
Co-authored-by: Jarsen <31397967+Jarsen136@users.noreply.github.com>
what about putting -- instead of the numbers? |
as you wish, I've used '--' instead of the IDs. |
and in the gallery item it should apply as well or not yet? |
I don't really know, but I guess your question was a request :D ? Please let me know if anything else is required here, I'm not sure if empty dashes work better than some ID. |
it was not :D I am not sure how it is with the linking to this collection etc. Your opinion? Id is just long, that was my point :D soo, lets leave the id there and then fix the option to mint nft without name :D |
Code Climate has analyzed commit dfb710c and detected 0 issues on this pull request. View more on Code Climate. |
I think I have fixed it elsewhere, although there may be some other problem I've seen conversation about. People won't be allowed to pass empty strings anymore, that should be clear now. Anyways, this PR is, I guess, done now. |
pay 20 usd |
😍 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?
Community participation
Screenshot 📸