Angular Code Review

Clase 5 de 5 • Prueba Técnica: Desarrollo Frontend con Angular

Kevin Fiorentino

student•

Gran experiencia TDD me llevo de este ejercicio. Comparto mi solución y la app desplegada en Vercel. Por más simple que parezca el ejercicio, siempre hay algún bloqueo y se aprende algo nuevo. Muchas gracias!

Diego Salcedo

student•

Para hacer el deploy a github pages instalen ng add angular-cli-ghpages y luego hagan el deploy con ng deploy --base-href=Pages

Para evitar el error de que aparezca la pantalla en blanco

Milton Estrada

student•

Gracias, pero cuando recargo me dice que 404 no sé si te paso?

Milton Estrada

student•

Por si a alguien le paso lo mismo que ami, esta fue la solución que encuentre.

- ng add angular-cli-ghpages
- ng build --configuration=production --base-href "https://<username>.github.io/<name-project>"
 - ngh --dir dist/yourProyect
----Example----
- ng build --configuration=production --base-href "https://miltondw.github.io/laboratorio-mydayapp-angular-solution/"
-  ngh --dir dist/myapp

MyApp

Diego Salcedo

student•

Para correr los test en el pull request me dice que debe aprobarlo. ¿No hay problema con esto?

Kevin Fiorentino

student•

Idem, también me aparece lo mismo. Supongo que Nico debe permitir que los test corran cuando es la primera vez que hacemos un PR.

Jesner Wilian Ramirez Cueva

student•

No me parece algo justo enseñar angular 17 y poner el proyecto en angular 15.

Diego Salcedo

student•

Listo! Luego de un par de horas, terminé el laboratorio y pasé todas las pruebas.

Pueden ver el resultado en https://playerdiego.github.io/lab-angular-platzi/

Benjamin Edilfonso

student•

José Luis Jiménez

student•

Todas mis pruebas corren bien en local..pero cuando hago el pull request al proyecto original..no se ejecuta los tests el servidor de integración continua. Me lanza este error:1 workflow awaiting approval..alguna idea?

José Gregorio López Arias

student•

Tengo un problema cuando cambio el path de la ruta por :filter ya no carga el componente Home al menos que le pase un valor en al url ejemplo localhost:4200/all asi tengo las ruta const routes: Routes = [ { path: ':filter', component: HomeComponent, } ];

constructor(private todoService: TodoService, private route: ActivatedRoute) { this.route.paramMap.subscribe((params) => { const filter = params.get('filter') as Filter; this.todoService.changeFilter(filter || 'all'); }); }

Andrés Medina

student•

Me parece interesante mencionar que la mayoría de las pruebas fallaron hasta que no apliqué la persistencia con localeStorage. Yo quería ir probando una a una, pero hasta no implementar la persistencia, las pruebas fallaban. 🤔

Max Andy Diaz Neyra

student•

Si, lo mismo me pasó. No implementaron bien la idea jajaja

ANDRES ALFONSO MIRA MEJIA

student•

✅

Oriana Stefany Bautista Iscala

student•

Solución: aplicación desplegada Repo: https://github.com/oriBautista99/laboratorio-mydayapp-angular/tree/technical-test

Carlos Gomez

student•

Me siento orgulloso de mi avance con estos cursos de Angular 😎, realice el test a conciencia y este fue el resultado:

MyDayApp - Angular (carlosgomez-99.github.io)

Sin embargo, 7 de las 20 pruebas no me quisieron pasar y no entiendo por qué, si alguien quiere revisar les dejo el repo:

CarlosGomez-99/PruebaTecnicaAngularPlatzi (github.com)

Nicolas Erazo Arce

student•

Mi solución desplegada! Enlace al sitio Quedo atento a comentarios y observaciones.

Alexis González

student•

Excelente ejercicio. Mi solución App desplegada en Netlify

Diego Salcedo

student•

Si no les aparece el mismo mensaje de Nicolas para hacer el compare & pull request. Pueden hacerlo desde Contribute > Open pull request

Franco Renato Carrillo Guevara

student•

He intentado de resolver el problema que indica ahi, pero no he podido. Hasta he replicado el testing, pero si realiza lo que "supuestamente" no esta cumpliendo :c

Aca les dejo el proyecto deployeado para que vean que digo la verdad :c https://my-todo-app-franchest.netlify.app

Santiago Martinez Martinez

student•

Hola Franco, revisa cómo tienes el routing, me parece que el testing sólo evalua la persistencia en la ruta "localhost:4200" sin / u otros parametros. Yo tuve ese problema ya que iniciaba la aplicación y lo enviaba directamente a "localhost:4200/all"

Victor Alfredo Matzar Say

student•

Wow, Gracias Santiago Martinez Martinez, quitando el parametro /all, me funciono :)

Angular Code Review

Preparación para la prueba

Setup de la prueba

Requisitos para presentar pruebas técnicas con Angular

Prueba técnica de Angular Developer

Solución de la prueba

¿Quieres más laboratorios con Angular?

Angular Code Review