FaztWeb / node-upload-images

14 stars 18 forks source link

Actualización en el paquete uuid #2

Open LAS1819 opened 4 years ago

LAS1819 commented 4 years ago

Hola, Fazt.

Ante todo darte la enhorabuena por los vídeos, guías y repositorios que compartes con todos nosotros, pues son muy didácticos y a mí, personalmente, me están sirviendo muchísimo de cara a mi carrera profesional.

Quería comentar que el módulo uuid ya no se requiere de la forma que la requieres en tu código: https://github.com/FaztWeb/node-upload-images/blob/043a3d5ec9e85adba4efce0abe046b7f7ae478c2/src/index.js#L6

Si no que ahora se requiere de esta forma: const { v4: uuidv4 } = require('uuid');

Al menos, a mí me daba error de la otra forma y he visto desde la página oficial de uuid que se importaba de esa manera.

Por tanto, cuando usamos la constante en esta línea: https://github.com/FaztWeb/node-upload-images/blob/043a3d5ec9e85adba4efce0abe046b7f7ae478c2/src/index.js#L38

Tendremos que cambiar uuid() por uuidv4()

Espero que sirva de ayuda.

Un saludo cordial y gracias de nuevo,

LuxuS7

NitTsu03 commented 2 years ago

Gracias, me daba ese error