Conversation
implement synopsis ,author,imgBook
implement synopsis,author,imgBook
…225/TFM-backend-amarillo into PabloPianelo/VISUALFICHA
array author and genre
…R0225/TFM-backend-amarillo into PabloPianelo/LOGIN-BACKEND
Pablo pianelo/login backend
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| @@ -0,0 +1,55 @@ | |||
| const Book = require('../models/BookModel'); | |||
|
|
|||
|
|
|||
There was a problem hiding this comment.
Elimina las líneas en blanco innecesarias al principio del archivo. Una sola línea de separación es suficiente.
| * GET /books | ||
| */ | ||
|
|
||
| const getBook = async (req, res) => { |
There was a problem hiding this comment.
El nombre de la función getBook debería ser getBooks ya que devuelve múltiples libros, no uno solo.
|
|
||
| const getBook = async (req, res) => { | ||
| try { | ||
|
|
There was a problem hiding this comment.
Elimina esta línea en blanco innecesaria dentro de la función.
| try { | ||
|
|
||
|
|
||
| const books = await Book.find(); // Obtén todos los libros |
There was a problem hiding this comment.
Considera añadir opciones de paginación para optimizar el rendimiento cuando la colección crezca. Por ejemplo: const books = await Book.find().limit(limit).skip(skip);
| } | ||
| }; | ||
|
|
||
|
|
There was a problem hiding this comment.
Elimina las líneas en blanco innecesarias entre funciones. Una sola línea de separación es suficiente.
| .matches(/[!@#$%^&*(),.?":{}|<>]/) | ||
| .withMessage('La contraseña debe contener al menos un carácter especial'),validationChecker | ||
| ] | ||
| ,loginUser ) |
There was a problem hiding this comment.
Mejora el formato de la ruta. Coloca loginUser en la misma línea que el cierre del array de validaciones o en una nueva línea con indentación adecuada.
| .withMessage('La contraseña debe contener al menos un carácter especial'),validationChecker | ||
| ] | ||
| ,loginUser ) | ||
| //solo puede estrar si esta logueado |
There was a problem hiding this comment.
Mejora el comentario para que sea más descriptivo. "solo puede estrar si esta logueado" contiene un error tipográfico y podría ser más formal, como: "// Middleware de autenticación - Las rutas siguientes requieren usuario autenticado".
| //solo puede estrar si esta logueado | ||
| router.use(getAuthUser); | ||
|
|
||
| //prueba |
There was a problem hiding this comment.
Elimina el comentario y código comentado. Si la ruta /me no está implementada, es mejor quitar estas líneas hasta que se necesiten.
| //prueba | ||
| // router.get('/me',getUser); | ||
|
|
||
|
|
There was a problem hiding this comment.
Elimina la línea en blanco excesiva antes de module.exports.
| // router.get('/me',getUser); | ||
|
|
||
|
|
||
| module.exports = router; No newline at end of file |
There was a problem hiding this comment.
Añade una nueva línea al final del archivo para seguir las convenciones estándar de archivos de texto.
2 participants
No description provided.