Skip to content

BCN-PT-Marc Valles#9

Open
efildriew wants to merge 5 commits into
ironhackbcn:masterfrom
efildriew:master
Open

BCN-PT-Marc Valles#9
efildriew wants to merge 5 commits into
ironhackbcn:masterfrom
efildriew:master

Conversation

@efildriew

Copy link
Copy Markdown

Buenos días.

Entiendo que me he ido mas allá de mis capacidades haciendo este ejercicio como lo he hecho. Después de estar toda la mañana, consigo que se borren las líneas YA CREADAS CON ANTERIORIDAD, consigo crear nuevas líneas y que las ponga donde debe ponerlas (primero las ponía después del botón de calcular precio...), pero no consigo que los nuevos botones de delete borren las líneas.

Acerca de calcular los precios, la verdad es que ahora mismo no veo como seguir. Ayer Alex me enseñó como grababa los valores de los inputs, he intentado hacerlo 40 veces y no me funciona, y no lo entiendo. Voy a descansar un rato de este ejercicio y volveré a él con la mente más fresca, me siento que me estoy dando contra la pared continuamente.

@efildriew

Copy link
Copy Markdown
Author

A ver, en teoría he hecho un nuevo Pull Request, pero no estoy seguro. Por si acaso...

Veamos, consigo que haga el total pero... bueno, mientras estaba escribiendo esto, he encontrado la solución XD. Voy a hacer otro!

@efildriew

Copy link
Copy Markdown
Author

Bueno, en primer lugar, veo que la he liado con el nombre del pull request. Yo pensaba que lo podría cambiar una vez hecho el definitivo, pero veo que no. Creo que debería cerrarlo y abrir otro, pero "at this point I'm too afraid to try". Ya lo hablamos en clase luego.

Ahora mismo lo único que le falta es conseguir borrar las nuevas líneas creadas. Voy a ver si me rompo el tarro un poco más y lo saco.

@efildriew

Copy link
Copy Markdown
Author

Se acabó. Ya he conseguido funcionalidad total (hasta que lo hagáis petar XD)

Luego comentamos si cierro este pull request y abro uno nuevo, o dejo este. Gracias por todo chicos!

@efildriew efildriew changed the title Menudo infierno BCN-PT-Marc Valles Jun 27, 2019
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.

1 participant