Skip to content
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

chore: migrate nuxt module to HMR version #3232

Merged

Conversation

Szymon-dziewonski
Copy link
Contributor

Related issue

Closes #

Scope of work

Newest version @nuxtjs/tailwindicss:6.12.1 that supports HMR our module has to migrated

Connected GH issue nuxt-modules/tailwindcss#898 (comment)

Screenshots of visual changes

Checklist

  • Self code-reviewed
  • Changes documented
  • Semantic HTML
  • SSR-friendly
  • Caching friendly
  • a11y for WCAG 2.0 AA
  • examples created
  • blocks created
  • cypress tests created

Copy link

changeset-bot bot commented Oct 7, 2024

🦋 Changeset detected

Latest commit: 79406be

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@storefront-ui/nuxt Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Szymon-dziewonski Szymon-dziewonski force-pushed the chore--migrate-nuxt-module-to-hmr-version branch from 46e682f to 885d863 Compare October 7, 2024 14:23
@Szymon-dziewonski Szymon-dziewonski force-pushed the chore--migrate-nuxt-module-to-hmr-version branch from 885d863 to 15d6ddb Compare October 7, 2024 14:41
"nuxt": "^3.12.4",
"nuxt-gtag": "^2.1.0"
"nuxt": "^3.13.2",
"nuxt-gtag": "^2.0.5"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the downgrade on purpose?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I havent changed this version of nuxt-gtag, weird, yarn had to change it, good catch will check it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +1 to +4
// How to test? check if variable is overwritten with current priority table
// 1. `tailwind.config.ts` file
// 2. `nuxt.config.ts` file with `tailwindcss` property
// 3. default configuration inside module
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Szymon-dziewonski Szymon-dziewonski merged commit 30bfa9a into v2-develop Oct 8, 2024
16 checks passed
@Szymon-dziewonski Szymon-dziewonski deleted the chore--migrate-nuxt-module-to-hmr-version branch October 8, 2024 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants