Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🛠️ Refactorización de question-service.js y Modularización del Código 📦 #49

Merged
merged 2 commits into from
Feb 27, 2024

Conversation

Verzidee
Copy link
Contributor

Originalmente, todo el código de generación de preguntas estaba confinado en un único archivo "question-service.js", abarcando desde la lógica de la aplicación Express hasta la ejecución de consultas SPARQL en Wikidata y el manejo de las respuestas.

Cambios realizados:

  1. 📚 Separación de la lógica SPARQL: Extraje la lógica para realizar consultas SPARQL a Wikidata fuera de "question-service.js". Ahora, ésta reside en "wikiQuery.js" y "wikiCall.js". "wikiQuery" se encarga de construir las consultas SPARQL y manejar las respuestas de la API, mientras que "wikiCall.js" se ocupa de ejecutar llamadas a la API de Wikidata.

  2. 🧩 Introducción de plantillas de preguntas: Implemente un nuevo módulo "questionTemplates.js" que define plantillas para generar distintos tipos de preguntas, facilitando la creación de nuevos tipos de preguntas.

  3. 🔄 Modularización y Reutilización de Código: Esta reestructuración ha mejorado la reutilización de funciones comunes, como las llamadas a la API de Wikidata o la selección aleatoria de elementos, reduciendo la duplicación de código.

  4. 🚀 Cambio en el Endpoint del gateway: Renombre el endpoint para generar preguntas de /generatequestion a /getquestion, para alinear mejor con la funcionalidad esperada por la interfaz.

Copy link
Contributor

@Santiago21112001 Santiago21112001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

He puesto el docker y package del master para que fuera un microservicio el questionservice. Tiene la interfaz Game

@Santiago21112001 Santiago21112001 merged commit 3f342fa into develop Feb 27, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants