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

fix: drop vueI18n option warning #2324

Merged
merged 1 commit into from
Aug 17, 2023
Merged

Conversation

kazupon
Copy link
Collaborator

@kazupon kazupon commented Aug 17, 2023

πŸ”— Linked issue

❓ Type of change

  • πŸ“– Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

πŸ“ Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@kazupon kazupon merged commit d222364 into next Aug 17, 2023
8 checks passed
@kazupon kazupon deleted the fix/drop-vue-i18n-options-warning branch August 17, 2023 05:56
@WolfgangDrescher
Copy link

Is there a reason why this warning was removed? It took me a while to debug this from a project I did not update for several months but that was using @nuxtjs/i18n-edge version 8.0.0-beta.10-28003311.2c39470 in combination with the old i18n.vueI18n config and today updated to 8.0.0-rc.4-28234551.f63ee58. This warning would have helped me a lot.

@WolfgangDrescher
Copy link

The error message input.includes is not a function was thrown, and it wasn't really clear where the problem was.

 ERROR  input.includes is not a function

  at normalizeWindowsPath (node_modules/pathe/dist/shared/pathe.92c04245.mjs:2:24)
  at normalize (node_modules/pathe/dist/shared/pathe.92c04245.mjs:17:10)
  at resolvePath (node_modules/@nuxt/kit/dist/index.mjs:2340:10)
  at resolveVueI18nConfigInfo (node_modules/@nuxtjs/i18n-edge/dist/module.mjs:243:43)
  at setup (node_modules/@nuxtjs/i18n-edge/dist/module.mjs:1439:41)
  at async normalizedModule (node_modules/@nuxt/kit/dist/index.mjs:2141:17)
  at async installModule (node_modules/@nuxt/kit/dist/index.mjs:2426:95)
  at async initNuxt (node_modules/nuxt/dist/index.mjs:3533:7)
  at async loadNuxt (node_modules/nuxt/dist/index.mjs:3616:5)
  at async loadNuxt (node_modules/@nuxt/kit/dist/index.mjs:2571:19)

DarthGigi pushed a commit to DarthGigi/i18n that referenced this pull request Apr 16, 2024
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.

2 participants