Comentarios generales del proyecto#4
Open
darodriguez15 wants to merge 7 commits intoeacabello10:jairofrom
Open
Comentarios generales del proyecto#4darodriguez15 wants to merge 7 commits intoeacabello10:jairofrom
darodriguez15 wants to merge 7 commits intoeacabello10:jairofrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Me parece acertado la creación de componentes que hacen tareas específicas, lo que ayuda en la modularidad del proyecto, aunque sería muy útil si los metieras en carpetas separadas para que fue más fácil de entender para un tercero.
No es tanto como comentario sino para aprender algo nuevo, para que utilizaste los archivos.jade? si react se encarga de manejar el front ?
Muy útil que las funciones en donde llamas la base de datos las implementaste con funciones no bloqueantes (callback) para que tu app pueda hacer más de una actividad al tiempo.
Sería bueno que poblaras más tu base de datos, ya que solo muestra 3 juegos.
Deberías trabajarle un poco al diseño de la página ya que se vería muy bien una foto del juego o un gameplay y que la organización sea mas agradable para el usuario.
Sería bueno que en README incluyeras una descripción de que hace en si el proyecto, ya que para alguien que no sepa el tema en concreto puede creer que puede ser una venta, un alquiler, un foro. Generando asi ambigüedad en el propósito del proyecto
La idea del proyecto me parecio excelente y deberías meterle la ficha a la interfaz de usuario, por que sin duda alguna yo utilizaría el proyecto.