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

some improvements for cipher project #1289

Merged
merged 2 commits into from
Jan 26, 2023

Conversation

ssinuco
Copy link
Collaborator

@ssinuco ssinuco commented Jan 10, 2023

Propongo los siguientes cambios:

  • En la seccion Preámbulo mejorar la explicacion sobre qué es cifrar
  • En la seccion "8. Pistas, tips y lecturas complementarias", subseccion "Prepara tu PC para trabajar":
  1. Eliminar referencia a Atom porque ya esta deprecated
  2. Eliminar referencia a Windows Subsystem for Linux. Propongo eliminarlo porque las estudiantes suelen confundirse con esto e intentan configurar el WSL junto con Cmder y Git bash. Considero que es mejor si les ahorramos esta confucion. Por otro lado, en mi experiencia personal, WSL suele ser exigente para la máquina y supongo que las estudiantes que no tengan un computador bueno tendrian problemas. Y finalmente me parece un poco exagerado configurar todo WSL para algo sencillo como el primer proyecto. Dejo abierta la discusion.
  3. En la parte que habla de hacer un fork decia "La otra integrante del equipo deber hacer un fork " pero este proyecto es individual, entonces lo modifiqué.
  4. En la seccion "Recursos y temas relacionados", elimine el link "Documentación de NPM" porque no es necseario revisarla para completar el proyecto.

Pendiente:

  • Hacer estos mismo cambios en la version de portugués

@mfdebian mfdebian self-requested a review January 10, 2023 20:25
@mfdebian mfdebian self-assigned this Jan 10, 2023
@mfdebian mfdebian added enhancement New feature or request idea Ideas, sugerencias, comentarios generales y feedback content Relacionado al contenido de proyectos y tópicos labels Jan 10, 2023
@mfdebian mfdebian added this to the v5.6.0 milestone Jan 10, 2023
@mfdebian
Copy link
Collaborator

Nuevamente gracias!!! @ssinuco 🙌

@@ -297,7 +293,6 @@ Desarrollo Front-end:
* [Aprende más sobre `ASCII`](http://conceptodefinicion.de/ascii/)
* Aprende más sobre [objetos](https://es.javascript.info/object) y
[como definir métodos](https://developer.mozilla.org/es/docs/Web/JavaScript/Guide/Working_with_Objects#definici%C3%B3n_de_m%C3%A9todos)
* [Documentación de NPM](https://docs.npmjs.com/)
Copy link
Member

Choose a reason for hiding this comment

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

🤪 no se porque este estaba

unjust
unjust previously approved these changes Jan 10, 2023
Copy link
Member

@unjust unjust left a comment

Choose a reason for hiding this comment

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

LGTM! pero el md-lint tiene problemas ;) y falta portugues

@ssinuco
Copy link
Collaborator Author

ssinuco commented Jan 12, 2023

LGTM! pero el md-lint tiene problemas ;) y falta portugues

@unjust @mfdebian listo el md-lint y la version de portuges!

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 enhancement New feature or request idea Ideas, sugerencias, comentarios generales y feedback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants