HiginioUrbano / practica_dwec_gestor_presupuesto

Práctica de Gestión de Presupuestos para el módulo de Desarrollo Web en Entorno Cliente
GNU General Public License v3.0
0 stars 0 forks source link

Borrar Etiquetas #10

Closed HiginioUrbano closed 2 years ago

HiginioUrbano commented 2 years ago

@pedroprieto No consigo que me pase el test final de borrar etiquetas.

He probado de todo, usar split, pusar .gasto sin .gasto. En la funcion dentro del bucle he probado también de todo y no se que detalle me falte, creo que he probado todo lo posible y que se me ha ocurrido, pero ni por suerte he conseguido dar con la tecla. Tengo ambas partes mal? la función consstructuro y el botón del span o solo una de las dos? También he mirado a través de la consola del navegador, pero no aparece nada esta vez.

pedroprieto commented 2 years ago

https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/blob/fa2325df1c18166f201489f004fda15f2cfab894/js/gestionPresupuestoWeb.js#L40

Tienes que pasar también la etiqueta como propiedad del objeto. Igual que pasas el gasto, tienes que pasar la etiqueta.

Y luego, en la función manejadora: https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/blob/fa2325df1c18166f201489f004fda15f2cfab894/js/gestionPresupuestoWeb.js#L156

Tienes que hacer referencia al parámetro this.etiqueta, que es la que quieres borrar. Tal como estás haciendo estás tratando de borrar todas las del gasto. Y encima no le pasas bien el parámetro, porque le pasas un array, no una lista.

Un saludo!

HiginioUrbano commented 2 years ago

Si, si esas cosas las habia probado ya, pero no me iban igualmente, te habia hecho el push con lo ultimo que habia probado, te actualizo con un issue y me dices donde estoy fallando.

El jue, 11 nov 2021 a las 16:16, Pedro Prieto @.***>) escribió:

https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/blob/fa2325df1c18166f201489f004fda15f2cfab894/js/gestionPresupuestoWeb.js#L40

Tienes que pasar también la etiqueta como propiedad del objeto. Igual que pasas el gasto, tienes que pasar la etiqueta.

Y luego, en la función manejadora:

https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/blob/fa2325df1c18166f201489f004fda15f2cfab894/js/gestionPresupuestoWeb.js#L156

Tienes que hacer referencia al parámetro this.etiqueta, que es la que quieres borrar. Tal como estás haciendo estás tratando de borrar todas las del gasto. Y encima no le pasas bien el parámetro, porque le pasas un array, no una lista.

Un saludo!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/issues/10#issuecomment-966385338, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUM3WNPXNX7ZLIVQRVZLXRTULPM6BANCNFSM5H2RJADA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

pedroprieto commented 2 years ago

Reviso el otro issue.