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

Agrega codigo necesario por mongoDB setup y spawn en Windows #1353

Merged
merged 55 commits into from
May 17, 2023

Conversation

unjust
Copy link
Member

@unjust unjust commented Apr 3, 2023

Un primer paso para entregar el proyecto con el trabajo necesario para configurar MongoDB con los e2e tests.
Tambien cambie una linea para arreglar spawn en windows (que no funciona por una estudiante en windows)

Ya tenemos una guia "Getting Started" que habla mucho de docker. Muchas estudiantes que usan Windows deciden hacer el project sin Docker, porque Docker + Windows toman muchos recursos y no es una buena experiencia (he escuchado).

Para mejorar la experiencia de proyecto:

  • Optamos elegir y entregar boilerplate con MongoDB, con codigo en el globalSetup de e2e para funcionar xon mongo memory server y jest
  • Escribi una guia aparte para empezar con MongoDB, que describe la instalación local, los ToDos para implementar y al final habla de config de e2e tests
  • Dejamos docker pero como Hacker edition

Proyecto para probar https://github.com/Laboratoria/CURR-burger-queen-api

Por favor que revisamos:

Falta:

  • Prueba con Windows
  • Translation pt

@unjust unjust marked this pull request as draft April 4, 2023 15:41
mfdebian
mfdebian previously approved these changes Apr 4, 2023
@unjust unjust self-assigned this Apr 19, 2023
@unjust unjust added this to the 6.3 milestone Apr 19, 2023
@unjust unjust added the content Relacionado al contenido de proyectos y tópicos label Apr 19, 2023
@unjust unjust linked an issue Apr 19, 2023 that may be closed by this pull request
@unjust unjust marked this pull request as ready for review April 20, 2023 23:30
Copy link
Contributor

@merunga merunga left a comment

Choose a reason for hiding this comment

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

@unjust principalmente temas de idioma

esta instancia (por defecto `http://127.0.0.1:8080`). Esto asume que la base de
datos está disponible.

Los pruebas e2e depende en codigo de (`globalSetup.js`)[../e2e/globalSetup.js]
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Los pruebas e2e depende en codigo de (`globalSetup.js`)[../e2e/globalSetup.js]
Los pruebas e2e dependen del código de (`globalSetup.js`)[../e2e/globalSetup.js]

@unjust
Copy link
Member Author

unjust commented May 9, 2023

Despues mergear, quiero experimentar si este funciona

https://www.stevemar.net/github-code-in-readme/

pero con link no de mi repo pero de Laboratoria

https://github.com/unjust/bootcamp/blob/fce3433217f539aba6aa35936ddd98a1d2e97b5f/projects/04-burger-queen-api/config.js#L2

unjust and others added 2 commits May 16, 2023 14:06
Co-authored-by: Alfredo González <12631491+mfdebian@users.noreply.github.com>
Co-authored-by: Alfredo González <12631491+mfdebian@users.noreply.github.com>
@unjust unjust merged commit e918faf into Laboratoria:main May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Relacionado al contenido de proyectos y tópicos
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Configurar BQ API para ser mas facil correr los e2e
3 participants