jellyfin / jellyfin-vue

A modern web client for Jellyfin based on Vue
https://jellyfin.org
GNU General Public License v3.0
1.26k stars 226 forks source link

Error on npm start: details.family.toLowerCase is not a function #1864

Closed laedit closed 1 year ago

laedit commented 1 year ago

Description of the bug

Hi, I wanted to take a look at this promising project but encounter an error when following the Getting Started part of the readme:

 FATAL  details.family.toLowerCase is not a function                                                          10:52:23

  at C:\Users\jerem\Downloads\jellyfin-vue\node_modules\ip\lib\ip.js:385:39
  at Array.filter (<anonymous>)
  at C:\Users\jerem\Downloads\jellyfin-vue\node_modules\ip\lib\ip.js:384:37
  at Array.map (<anonymous>)
  at ip.address (C:\Users\jerem\Downloads\jellyfin-vue\node_modules\ip\lib\ip.js:379:37)
  at Listener.computeURL (C:\Users\jerem\Downloads\jellyfin-vue\node_modules\@nuxt\server\dist\server.js:469:60)
  at Listener.listen (C:\Users\jerem\Downloads\jellyfin-vue\node_modules\@nuxt\server\dist\server.js:510:10)
  at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
  at async Server.listen (C:\Users\jerem\Downloads\jellyfin-vue\node_modules\@nuxt\server\dist\server.js:973:5)
  at async Object._listenDev (C:\Users\jerem\Downloads\jellyfin-vue\node_modules\@nuxt\cli\dist\cli-dev.js:86:5)

Did I miss something?

Steps to reproduce

Follow the getting started part of the readme:

Expected behavior

No error and the server is up and the application ready to be admired.

Logs

No response

Screenshots

No response

Platform

Windows

Browser

Firefox

Jellyfin server version

10.8.9

Additional context

No response

ferferga commented 1 year ago

First of all, welcome here! 👋

Yes, it could be said that current master is in maintenance, so if you want to start contributing, please head over to the vite branch where we are working in upgrading this client to Vue 3 (which does have a single warning in npm i, but we're ignoring it for now because we want to update all the dependencies as soon as it reaches master and not in-between, in order to not pollute the commit history).

If you want to stick to current master, Windows is building fine, as you can check in our CI though. First thing I'd try is to check Node version is between the bounds we mention in Readme.

EDIT: If you don't plan to do any changes to it, just try out the demo link, you can connect to your server that way too.

laedit commented 1 year ago

So sorry, I haven't checked the requirement with enough attention.