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

Update Instant data missing from previous Commit #5

Merged
merged 2 commits into from
May 9, 2022

Conversation

ad-ha
Copy link
Contributor

@ad-ha ad-ha commented May 3, 2022

No description provided.

@ad-ha
Copy link
Contributor Author

ad-ha commented May 3, 2022

En el pull anterior se me olvidó añadir las líneas de actualización del atributo. El patch-1 y patch-2 van juntos.

@ldotlopez
Copy link
Owner

Hola Álvaro,

¿Podemos unir el PR #4 y PR #5 sobre un único parche y a partir de ahí lo miramos?

A mi me ha tocado hacer algún experimento porque no lo tenía muy claro, te dejo lo que he hecho:

git checkout patch-1 # Cambiar al patch-1, PR4
git rebase patch-2   # Aplicar los cambios de patch-2 sobre la rama actual, patch-1
git push             # Subirá los cambios (los dos commits) a tu rama patch-1, el PR #5 

Así cerramos este PR y continuamos en el otro

@ldotlopez ldotlopez self-assigned this May 5, 2022
@ad-ha
Copy link
Contributor Author

ad-ha commented May 5, 2022

Buenas

Intentaré hacerlo. Que sólo uso Git en la web. Si no cierro los dos pull y creo uno nuevo con todo junto.

@ad-ha
Copy link
Contributor Author

ad-ha commented May 5, 2022

Creo que está. El PR #4 creo que se puede cerrar.

@ldotlopez
Copy link
Owner

Gracias Álvaro.

Me gustaría comprobar si puedo extraer más datos del API de i-de además de las lecturas acumulada y real para empaquetarlo todo en un solo atributo… pero como veo que no estoy pudiendo sacar tiempo apruebo el PR y ya lo miraré.

@ldotlopez ldotlopez merged commit 5fe0482 into ldotlopez:main May 9, 2022
@ad-ha
Copy link
Contributor Author

ad-ha commented May 10, 2022

Hola Luis,
Gracias a ti. Lo vamos mirando.
Mientras, si te parece, abriré nuevos "issues" con las propuestas de desarrollos que me han ido pidiendo en mi integración y lo comentamos.
Saludos

@ad-ha ad-ha deleted the patch-2 branch May 10, 2022 22:48
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.

None yet

2 participants