cesarvaldezdev / hackersArena-API

The API used for hackersArena.
2 stars 0 forks source link

API // Revisión de primer entrega #54

Closed nmicht closed 5 years ago

nmicht commented 5 years ago

En cada archivo podrán encontrar comentarios taggeados con FIXME para que puedan revisar los detalles de su proyecto y trabajar en las correciones.

Además de los detalles en la lógica y código del proyecto, deben trabajar en las siguientes correcciones:

- Agregar documentacion a clase y metodos __FIXME__ [db/index.js](db/index.js)
- agregar documentacion a clase y métodos __FIXME__ [controllers/language.js](controllers/language.js)
- agregar documentacion a clase y métodos __FIXME__ [controllers/user.js](controllers/user.js)
- agregar documentacion a clase y métodos __FIXME__ [controllers/verdict.js](controllers/verdict.js)
- agregar documentacion a clase y métodos __FIXME__ [models/country.js](models/country.js)
- agregar documentacion a clase y métodos __FIXME__ [controllers/country.js](controllers/country.js)
- agregar documentacion a clase y métodos __FIXME__ [models/verdict.js](models/verdict.js)
- agregar documentacion a clase y métodos __FIXME__ [models/user.js](models/user.js)
- agregar documentacion a clase y métodos __FIXME__ [models/university.js](models/university.js)
- agregar documentacion a clase y métodos __FIXME__ [controllers/university.js](controllers/university.js)
- agregar documentacion a clase y métodos __FIXME__ [controllers/contest.js](controllers/contest.js)
- agregar documentacion a clase y métodos __FIXME__ [models/solution.js](models/solution.js)
- agregar documentacion a clase y métodos __FIXME__ [models/category.js](models/category.js)
- agregar documentacion a clase y métodos __FIXME__ [models/responses.js](models/responses.js)
- agregar documentacion a clase y métodos __FIXME__ [controllers/category.js](controllers/category.js)
- agregar documentacion a clase y métodos __FIXME__ [controllers/problem.js](controllers/problem.js)
- agregar documentacion a clase y métodos __FIXME__ [models/problem.js](models/problem.js)
- agregar documentacion a clase y métodos __FIXME__ [models/contest.js](models/contest.js)
- agregar documentacion a clase y métodos __FIXME__ [models/language.js](models/language.js)
- agregar documentacion a clase y métodos __FIXME__ [controllers/solution.js](controllers/solution.js)
- agregar documentacion a clases y métodos __FIXME__ [middlewares/validator.js](middlewares/validator.js)
- Agregar manejo de errores __FIXME__ [controllers/university.js](controllers/university.js)
- Agregar manejo de errores __FIXME__ [controllers/category.js](controllers/category.js)
- Agregar manejo de errores __FIXME__ [controllers/verdict.js](controllers/verdict.js)
- Agregar manejo de errores __FIXME__ [controllers/country.js](controllers/country.js)
- Agregar manejo de errores __FIXME__ [controllers/category.js](controllers/category.js)
- Agregar manejo de errores __FIXME__ [controllers/country.js](controllers/country.js)
- Agregar manejo de errores __FIXME__ [controllers/university.js](controllers/university.js)
- Agregar manejo de errores __FIXME__ [controllers/country.js](controllers/country.js)
- Agregar manejo de errores __FIXME__ [controllers/verdict.js](controllers/verdict.js)
- Agregar manejo de errores __FIXME__ [controllers/university.js](controllers/university.js)
- Agregar manejo de errores __FIXME__ [controllers/country.js](controllers/country.js)
- Agregar manejo de errores __FIXME__ [controllers/user.js](controllers/user.js)
- Agregar manejo de errores __FIXME__ [controllers/category.js](controllers/category.js)
- Agregar manejo de errores __FIXME__ [controllers/user.js](controllers/user.js)
- Agregar manejo de errores __FIXME__ [controllers/language.js](controllers/language.js)
- Agregar manejo de errores __FIXME__ [controllers/solution.js](controllers/solution.js)
- Agregar manejo de errores __FIXME__ [controllers/language.js](controllers/language.js)
- Agregar manejo de errores __FIXME__ [controllers/user.js](controllers/user.js)
- Agregar manejo de errores __FIXME__ [controllers/language.js](controllers/language.js)
- Agregar manejo de errores __FIXME__ [controllers/verdict.js](controllers/verdict.js)
- Agregar manejo de errores __FIXME__ [controllers/contest.js](controllers/contest.js)
- Agregar manejo de errores __FIXME__ [controllers/language.js](controllers/language.js)
- Agregar manejo de errores __FIXME__ [controllers/solution.js](controllers/solution.js)
- Agregar manejo de errores __FIXME__ [controllers/category.js](controllers/category.js)
- Agregar manejo de errores __FIXME__ [controllers/contest.js](controllers/contest.js)
- Agregar manejo de errores __FIXME__ [controllers/problem.js](controllers/problem.js)
- Agregar manejo de errores __FIXME__ [controllers/verdict.js](controllers/verdict.js)
- Agregar manejo de errores __FIXME__ [controllers/problem.js](controllers/problem.js)
- Agregar manejo de errores __FIXME__ [controllers/contest.js](controllers/contest.js)
- Agregar manejo de errores __FIXME__ [controllers/problem.js](controllers/problem.js)
- Agregar manejo de errores __FIXME__ [controllers/university.js](controllers/university.js)
- Agregar manejo de errores __FIXME__ [controllers/solution.js](controllers/solution.js)
- Agregar manejo de errores __FIXME__ [controllers/problem.js](controllers/problem.js)
- Agregar manejo de errores __FIXME__ [controllers/user.js](controllers/user.js)
- Agregar manejo de errores __FIXME__ [controllers/solution.js](controllers/solution.js)
- Agregar manejo de errores __FIXME__ [controllers/contest.js](controllers/contest.js)
- arreglar para mantener funciones en multiples lineas __FIXME__ [middlewares/validator.js](middlewares/validator.js)
- Corregir errores de linter __FIXME__ [routes/categories.js](routes/categories.js)
- Corregir errores de linter __FIXME__ [models/language.js](models/language.js)
- Corregir errores de linter __FIXME__ [models/problem.js](models/problem.js)
- Corregir errores de linter __FIXME__ [models/responses.js](models/responses.js)
- Corregir errores de linter __FIXME__ [models/contest.js](models/contest.js)
- Corregir errores de linter __FIXME__ [models/solution.js](models/solution.js)
- Corregir errores de linter __FIXME__ [models/university.js](models/university.js)
- Corregir errores de linter __FIXME__ [models/category.js](models/category.js)
- Corregir errores de linter __FIXME__ [controllers/university.js](controllers/university.js)
- Corregir errores de linter __FIXME__ [controllers/language.js](controllers/language.js)
- Corregir errores de linter __FIXME__ [models/user.js](models/user.js)
- Corregir errores de linter __FIXME__ [controllers/solution.js](controllers/solution.js)
- Corregir errores de linter __FIXME__ [models/verdict.js](models/verdict.js)
- Corregir errores de linter __FIXME__ [controllers/country.js](controllers/country.js)
- Corregir errores de linter __FIXME__ [models/country.js](models/country.js)
- Corregir errores de linter __FIXME__ [middlewares/validator.js](middlewares/validator.js)
- Corregir errores de linter __FIXME__ [routes/contests.js](routes/contests.js)
- Corregir errores de linter __FIXME__ [routes/countries.js](routes/countries.js)
- Corregir errores de linter __FIXME__ [routes/index.js](routes/index.js)
- Corregir errores de linter __FIXME__ [controllers/user.js](controllers/user.js)
- Corregir errores de linter __FIXME__ [routes/languages.js](routes/languages.js)
- Corregir errores de linter __FIXME__ [controllers/contest.js](controllers/contest.js)
- Corregir errores de linter __FIXME__ [routes/problems.js](routes/problems.js)
- Corregir errores de linter __FIXME__ [routes/solutions.js](routes/solutions.js)
- Corregir errores de linter __FIXME__ [routes/universities.js](routes/universities.js)
- Corregir errores de linter __FIXME__ [routes/users.js](routes/users.js)
- Corregir errores de linter __FIXME__ [routes/verdicts.js](routes/verdicts.js)
- Corregir errores de linter __FIXME__ [controllers/category.js](controllers/category.js)
- Corregir errores de linter __FIXME__ [controllers/verdict.js](controllers/verdict.js)
- Corregir errores de linter __FIXME__ [controllers/problem.js](controllers/problem.js)
- deben incluir el manejo de env __FIXME__ [app.js](app.js)
- falta validar el parametro __FIXME__ [routes/categories.js](routes/categories.js)
- falta validar el parametro __FIXME__ [routes/verdicts.js](routes/verdicts.js)
- falta validar el parametro __FIXME__ [routes/verdicts.js](routes/verdicts.js)
- falta validar el parametro __FIXME__ [routes/verdicts.js](routes/verdicts.js)
- falta validar el parametro __FIXME__ [routes/users.js](routes/users.js)
- falta validar el parametro __FIXME__ [routes/users.js](routes/users.js)
- falta validar el parametro __FIXME__ [routes/users.js](routes/users.js)
- falta validar el parametro __FIXME__ [routes/universities.js](routes/universities.js)
- falta validar el parametro __FIXME__ [routes/universities.js](routes/universities.js)
- falta validar el parametro __FIXME__ [routes/universities.js](routes/universities.js)
- falta validar el parametro __FIXME__ [routes/solutions.js](routes/solutions.js)
- falta validar el parametro __FIXME__ [routes/solutions.js](routes/solutions.js)
- falta validar el parametro __FIXME__ [routes/solutions.js](routes/solutions.js)
- falta validar el parametro __FIXME__ [routes/problems.js](routes/problems.js)
- falta validar el parametro __FIXME__ [routes/problems.js](routes/problems.js)
- falta validar el parametro __FIXME__ [routes/problems.js](routes/problems.js)
- falta validar el parametro __FIXME__ [routes/languages.js](routes/languages.js)
- falta validar el parametro __FIXME__ [routes/languages.js](routes/languages.js)
- falta validar el parametro __FIXME__ [routes/languages.js](routes/languages.js)
- falta validar el parametro __FIXME__ [routes/countries.js](routes/countries.js)
- falta validar el parametro __FIXME__ [routes/countries.js](routes/countries.js)
- falta validar el parametro __FIXME__ [routes/countries.js](routes/countries.js)
- falta validar el parametro __FIXME__ [routes/contests.js](routes/contests.js)
- falta validar el parametro __FIXME__ [routes/contests.js](routes/contests.js)
- falta validar el parametro __FIXME__ [routes/contests.js](routes/contests.js)
- falta validar el parametro __FIXME__ [routes/categories.js](routes/categories.js)
- falta validar el parametro __FIXME__ [routes/categories.js](routes/categories.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/contest.js](controllers/contest.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/user.js](controllers/user.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/verdict.js](controllers/verdict.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/category.js](controllers/category.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/language.js](controllers/language.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/language.js](controllers/language.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/user.js](controllers/user.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/language.js](controllers/language.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/country.js](controllers/country.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/verdict.js](controllers/verdict.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/category.js](controllers/category.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/country.js](controllers/country.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/university.js](controllers/university.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/country.js](controllers/country.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/country.js](controllers/country.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/verdict.js](controllers/verdict.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/contest.js](controllers/contest.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/category.js](controllers/category.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/problem.js](controllers/problem.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/university.js](controllers/university.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/problem.js](controllers/problem.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/university.js](controllers/university.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/university.js](controllers/university.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/problem.js](controllers/problem.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/solution.js](controllers/solution.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/solution.js](controllers/solution.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/contest.js](controllers/contest.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/category.js](controllers/category.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/problem.js](controllers/problem.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/solution.js](controllers/solution.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/solution.js](controllers/solution.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/language.js](controllers/language.js)
- los mensajes de las respuestas del API deben ser en ingles __FIXME__ [controllers/contest.js](controllers/contest.js)
- NO utilizar condicionales en una sola linea __FIXME__ [controllers/language.js](controllers/language.js)
- NO utilizar condicionales en una sola linea __FIXME__ [controllers/solution.js](controllers/solution.js)
- NO utilizar condicionales en una sola linea __FIXME__ [controllers/problem.js](controllers/problem.js)
- NO utilizar condicionales en una sola linea __FIXME__ [controllers/user.js](controllers/user.js)
- NO utilizar condicionales en una sola linea __FIXME__ [controllers/category.js](controllers/category.js)
- NO utilizar condicionales en una sola linea __FIXME__ [controllers/contest.js](controllers/contest.js)
- NO utilizar condicionales en una sola linea __FIXME__ [controllers/verdict.js](controllers/verdict.js)
- NO utilizar condicionales en una sola linea __FIXME__ [controllers/university.js](controllers/university.js)
- NO utilizar condicionales en una sola linea __FIXME__ [controllers/country.js](controllers/country.js)
- para el manejo de estados 0, 1 y 2 sería mejor usar una constante definida con un nombre mas claro __FIXME__ [models/contest.js](models/contest.js)
- para el manejo de estados 0, 1 y 2 sería mejor usar una constante definida con un nombre mas claro __FIXME__ [models/language.js](models/language.js)
- para el manejo de estados 0, 1 y 2 sería mejor usar una constante definida con un nombre mas claro __FIXME__ [models/problem.js](models/problem.js)
- para el manejo de estados 0, 1 y 2 sería mejor usar una constante definida con un nombre mas claro __FIXME__ [models/solution.js](models/solution.js)
- para el manejo de estados 0, 1 y 2 sería mejor usar una constante definida con un nombre mas claro __FIXME__ [models/country.js](models/country.js)
- para el manejo de estados 0, 1 y 2 sería mejor usar una constante definida con un nombre mas claro __FIXME__ [models/university.js](models/university.js)
- para el manejo de estados 0, 1 y 2 sería mejor usar una constante definida con un nombre mas claro __FIXME__ [models/responses.js](models/responses.js)
- para el manejo de estados 0, 1 y 2 sería mejor usar una constante definida con un nombre mas claro __FIXME__ [models/verdict.js](models/verdict.js)
- para el manejo de estados 0, 1 y 2 sería mejor usar una constante definida con un nombre mas claro __FIXME__ [models/category.js](models/category.js)
- para el manejo de estados 0, 1 y 2 sería mejor usar una constante definida con un nombre mas claro __FIXME__ [models/user.js](models/user.js)
- put es intencionado para ediciones y para para creaciones, por lo cual al parecer el metodo del controlador esta mal __FIXME__ [routes/universities.js](routes/universities.js)
- put es intencionado para ediciones y para para creaciones, por lo cual al parecer el metodo del controlador esta mal __FIXME__ [routes/countries.js](routes/countries.js)
- put es intencionado para ediciones y para para creaciones, por lo cual al parecer el metodo del controlador esta mal __FIXME__ [routes/categories.js](routes/categories.js)
- put es intencionado para ediciones y para para creaciones, por lo cual al parecer el metodo del controlador esta mal __FIXME__ [routes/users.js](routes/users.js)
- put es intencionado para ediciones y para para creaciones, por lo cual al parecer el metodo del controlador esta mal __FIXME__ [routes/solutions.js](routes/solutions.js)
- put es intencionado para ediciones y para para creaciones, por lo cual al parecer el metodo del controlador esta mal __FIXME__ [routes/contests.js](routes/contests.js)
- put es intencionado para ediciones y para para creaciones, por lo cual al parecer el metodo del controlador esta mal __FIXME__ [routes/problems.js](routes/problems.js)
- put es intencionado para ediciones y para para creaciones, por lo cual al parecer el metodo del controlador esta mal __FIXME__ [routes/languages.js](routes/languages.js)
- put es intencionado para ediciones y para para creaciones, por lo cual al parecer el metodo del controlador esta mal __FIXME__ [routes/verdicts.js](routes/verdicts.js)
- quitar codigo comentado __FIXME__ [controllers/verdict.js](controllers/verdict.js)

Se ve bien su proyecto, solo les faltan varios detalles por arreglar