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

Conversation

@vikiival
Copy link
Member

As pointed out by @kkukelka, we had two date libraries, since we use date-fns in some func and luxon was used only for charts I removed luxon and added chartjs-adapter-date-fns

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Do a quick check before the merge.

PR type

  • Bugfix
  • Feature
  • Refactoring

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

@vikiival vikiival self-assigned this Jan 20, 2022
@netlify
Copy link

netlify bot commented Jan 20, 2022

✔️ Deploy Preview for koda-nuxt ready!

🔨 Explore the source changes: 09a60f9

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

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

@github-actions
Copy link
Contributor

You're Pull Request scored a 0.016 out of a possible +5 on the sentiment scale. Here's a gif representation of your PR:
Boo from Monsters Inc blinking and looking into space neutrally

@github-actions
Copy link
Contributor

Gif

@yangwao yangwao merged commit ec09807 into main Jan 20, 2022
@vikiival vikiival deleted the bye-luxon branch January 20, 2022 17:28
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.

3 participants