Skip to content
This repository was archived by the owner on Dec 5, 2025. It is now read-only.

Conversation

@kkukelka
Copy link
Member

@kkukelka kkukelka commented Jan 2, 2022

Thank you for your contribution to the KodaDot NFT gallery.

👇 _ Do a quick check before the merge.

PR type

  • Bugfix
  • Feature
  • Refactoring

Screenshot 2022-01-02 at 16 27 51

Screenshot 2022-01-02 at 16 05 37

Before submitting Pull Request, please make sure:

  • My contribution builds clean without any errors or warnings
  • I've merged recent default branch -- main and I've no conflicts
  • I've tried respect high code quality standards
  • I've didn't break any original functionality
  • I've posted screenshot of demonstrated change in this PR

Optional

  • I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
  • I've tested PR on mobile and everything works
  • I found edge cases

What's new?

Had issue bounty label ?

  • Fill up your KSM address: Payout

Community participation

Screenshot

  • My fix has changed something on UI, a screenshot for others, is best to understand changes.

@netlify
Copy link

netlify bot commented Jan 2, 2022

✔️ Deploy Preview for koda-nuxt ready!

🔨 Explore the source changes: 45289ca

🔍 Inspect the deploy log: https://app.netlify.com/sites/koda-nuxt/deploys/61d1cebd81385c0007c7a66f

😎 Browse the preview: https://deploy-preview-1659--koda-nuxt.netlify.app

@yangwao
Copy link
Member

yangwao commented Jan 2, 2022

hmm if we are changing logos everywhere, what's about favicon?

image

@kkukelka
Copy link
Member Author

kkukelka commented Jan 2, 2022

hmm if we are changing logos everywhere, what's about favicon?

image

yes, good catch. gonna append it to this PR. just give me a minute

@kkukelka
Copy link
Member Author

kkukelka commented Jan 2, 2022

@yangwao done
Screenshot 2022-01-02 at 17 08 38

@yangwao
Copy link
Member

yangwao commented Jan 2, 2022

@yangwao yangwao merged commit 860f412 into kodadot:main-nuxt Jan 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clear out errors from manifest.json

2 participants