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

Entrega sesión 10 Frontend #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Sotrux
Copy link

@Sotrux Sotrux commented Jun 8, 2024

No description provided.

Copy link
Contributor

@alvarotech alvarotech left a comment

Choose a reason for hiding this comment

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

Muy buen trabajo Germán. A destacar la conversación inicial, pedirle que te comente las dudas que tenga y el proceso detallado, y poder ir luego paso a paso. Y usar para ello el contenido que ya tienes, y no reinventar la rueda, en este caso las propias instrucciones que eran bastante claras y concisas.

Veo que has podido ir lidiando con los problemas de contexto de Cursor, y detectando por ejemplo que faltaba la lógica de conexión con backend para el paso 4: drag and drop. Es importante revisar siempre todo lo que entrega, y tener ese espíritu crítico apoyado en conocimiento y experiencia para saber cuándo faltan cosas (en este caso, que no hubiera ninguna función que llame a API).

Pregunta respecto a la pérdida de contexto...qué has probado para solucionarlo que te funcione? Sirve el comenzar una nueva conversación?

@Sotrux
Copy link
Author

Sotrux commented Jun 10, 2024

Lo que me ha sucedido es que se me olvida dar Crt-Enter para que agarre el codebase y algunas veces pierde el contexto, pero al hacer de nuevo la pregunta e incluir el codebase se ubica de nuevo. Lo que más me inquieta es cuando no toma el codebase completo y hay que decirle explícitamente que tenga en cuenta ciertos archivos.

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