Reactive Forms exercise#45
Reactive Forms exercise#45SebastianMoreyraUTN wants to merge 1 commit intoutnfrrottads:reactive-formsfrom
Conversation
| this.taskGroup = new FormGroup({ | ||
| description: new FormControl('',[Validators.required]), | ||
| url: new FormControl('',[Validators.required,Validators.pattern(urlRegex)]), | ||
| id: new FormControl('') |
There was a problem hiding this comment.
me parece que este se podria evitar, pero deberias cambiar la forma en que se hace la edicion tomando los valores del form y copiandolos a la task, me parece que seria mejor
| task.isCompleted = false; | ||
| this.add.emit(task); | ||
| description.value = ''; | ||
| if (this.taskGroup.controls.id.value) { |
There was a problem hiding this comment.
tal vez yo usaria dos componentes diferentes con una herencia y me fijaria en el paso anterior para ver si estoy en edicion o no o sea en la lista cuando agrego con el boton hago una cosa y uso un componente y en el otro caso uso otra. Inclusive se podria usar el mismo componente y con un poco de polimorfismo determinar si esta en edicion o no.
| description.value = ''; | ||
| if (this.taskGroup.controls.id.value) { | ||
| let task = new TodoItem(); | ||
| task.description = this.taskGroup.controls.description.value; |
There was a problem hiding this comment.
la copia de los valores es igual en las dos ramas del if, deberian ir afuera para evitar duplicaicon de codigo
aotaduy
left a comment
aotaduy
left a comment
There was a problem hiding this comment.
Te deje algunas observaciones generales, pero muy bueno!
3 participants
Profe hago el pull request del ejercicio de Reactive-Forms,
Saludos !