nuxt-modules / algolia

🔎 Algolia module for Nuxt
https://algolia.nuxtjs.org/
MIT License
191 stars 35 forks source link

Update 1.vue-instantsearch.md #93

Closed podlebar closed 2 years ago

podlebar commented 2 years ago

since RC9 autoImports:dirs is deprecated on some environments like Vercel the build fails with the old import statement. This version should work fine on all envs i tested.

Example Error:

2022-09-22T08:38:15.798Z    c0351ce2-5adc-45af-b96d-54647a71d8e6    ERROR   [nuxt] [request error] [unhandled] [500] Directory import '/var/task/node_modules/vue-instantsearch/vue3/es' is not supported resolving ES modules imported from /var/task/chunks/app/server.mjs
Did you mean to import vue-instantsearch/vue3/es/index.js?
  at new NodeError (node:internal/errors:372:5)  
  at finalizeResolution (node:internal/modules/esm/resolve:433:17)  
  at moduleResolve (node:internal/modules/esm/resolve:1009:10)  
  at defaultResolve (node:internal/modules/esm/resolve:1218:11)  
  at ESMLoader.resolve (node:internal/modules/esm/loader:580:30)  
  at ESMLoader.getModuleJob (node:internal/modules/esm/loader:294:18)  
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:80:40)  
  at link (node:internal/modules/esm/module_job:78:36)
netlify[bot] commented 2 years ago

Deploy Preview for nuxt-algolia-module ready!

Name Link
Latest commit 9b8bf2db061df07606b4380235734ae539567703
Latest deploy log https://app.netlify.com/sites/nuxt-algolia-module/deploys/632c1f93d85c4d000850ccd9
Deploy Preview https://deploy-preview-93--nuxt-algolia-module.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Baroshem commented 2 years ago

Hi @podlebar

Have you verified that when changing the path to this the module and instantsearch integration will work as planned? I had problems in the past when changing the path to the lib did not fix the issue even though terminal was recommending to do it.

podlebar commented 2 years ago

yes.. i did test it in a production environment where i use this module with Vue Instantsearch components.

Baroshem commented 2 years ago

Ok, thanks for that. Merging! :)