Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

Major codebase update #17

Open
wants to merge 35 commits into
base: master
Choose a base branch
from
Open

Major codebase update #17

wants to merge 35 commits into from

Conversation

mornir
Copy link

@mornir mornir commented Feb 2, 2021

  • Upgrade Nuxt to its latest version and resolve breaking changes.
  • Use new Nuxt crawler and full static mode
  • Use nuxt-sanity module, with its Portable Text and Image helpers.
  • Upgrade Snipcart from 2.X to 3.X. Use official nuxt-snipcart module.
  • Remove numeral package. Use native intl.numberFormat instead.
  • Various improvements (HTML semantics, Vue best practices, etc.)

I tried to hide my poor commit messages by squashing all my commits, but failed miserably 😅
If you agree to merge this PR, please use the "Squash and merge" option 😉

Otherwise don't hesitate to make/request changes.

P.S.: Happy to see these numbers 😁
image

Squashed commit:

[03cd73b] Add more information to readme

[dc4c19f] Update readme

[62fdee4] Fix list styling

[67a49f9] Remove unused component

[f1f2902] Code review

[1fe24e7] Update readme

[da2e251] Remove numeral, intl.numberformat instead

[0865aef] Update readme

[0eaff82] Lint readme

[a2cf47c] Run yarn lint

[d96a16a] Generate pages at build time

[9f880fd] Temp disable routes generation

[b316482] Yarn upgrade

[6c3db55] Remove unneeded ref

[bde88de] Add nuxt-snipcart module

[2791223] Use SanityContent and SanityImage

[eac8430] Add nuxt-sanity module

[4745c86] Fix nuxtServerInit

[e70d6c8] Add target static

[d33612b] Fix eslint config

[6c6d3bb] Use nuxt server init

[14f7883] Remove postcss

[500bcf5] Update all deps

[f3aceb6] Add jsconfig
@mornir
Copy link
Author

mornir commented Feb 8, 2021

Hi @vicbergquist
I was wondering if you would have the time to look at this PR. I'm asking you because I know that you have experience with Nuxt 😊
You can check out to preview deploy on Netlify to see if everything is working properly.

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.

1 participant