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

[NEW] Añadida nueva retención de arrendamientos v7.0 #204

Closed
wants to merge 5 commits into from
Closed

[NEW] Añadida nueva retención de arrendamientos v7.0 #204

wants to merge 5 commits into from

Conversation

cubells
Copy link
Sponsor Member

@cubells cubells commented Sep 8, 2015

Hecho @pedrobaeza

@cubells cubells changed the title [NEW] Añadida nueva retención de arrendamientos [NEW] Añadida nueva retención de arrendamientos v7.0 Sep 8, 2015
@cubells
Copy link
Sponsor Member Author

cubells commented Sep 8, 2015

No había aplicado los cambios de 01d3ad6

<field name="description">S_RAC195A</field>
<field name="type_tax_use">sale</field>
<field name="account_paid_id" ref="pgc_473_child"/>
<field name="name">Retenciones a cuenta IRPF 19,5% (Arrendamientos)</field>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quita por favor la palabra IRPF de aquí

@pedrobaeza
Copy link
Member

Y si puedes juntar todos los commits en 1, mucho mejor 😉 !

@cubells
Copy link
Sponsor Member Author

cubells commented Sep 9, 2015

Pues va a ser que no me aclaro haciendo el rebase.

@pedrobaeza
Copy link
Member

Tienes que hacer rebase -i 221eea2445abecdf7128824603ffa3b4fb6b3823 y en el editor que te aparece con los 5 commits actuales, cambiar la primera palabra a squash de los 4 últimos y dejar el primero tal cual. Así, se juntarán todos los commits en uno.

@cubells
Copy link
Sponsor Member Author

cubells commented Sep 9, 2015

No me deja Pedro:
Could not apply e2193a6e97ca74e6fb0ce8157a60defc71330667... [NEW] Añadida nueva retención de arrendamientos

Yo lo hacía con:

git rebase -i HEAD~4

pero no sé qué he hecho o si es porque hay un commit automático de la traducción posterior a los míos que no me deja. Podría forzar pero no quiero cagarla más.

Tú dirás qué hago.

@pedrobaeza
Copy link
Member

Yo te diría que partieras de una rama 7.0 tal cual, copies estos cambios (los archivos), y hagas un nuevo commit.

@cubells
Copy link
Sponsor Member Author

cubells commented Sep 9, 2015

Sí, es lo más fàcil. Cierra el PR que voy.

@pedrobaeza
Copy link
Member

No hace falta cerrarlo. Haz push con el mismo nombre de rama y el argumento -f.

@cubells
Copy link
Sponsor Member Author

cubells commented Sep 9, 2015

Nada. Algo tengo mal. Lo siento.

@pedrobaeza
Copy link
Member

Ok, haz otro PR y ya está.

@pedrobaeza pedrobaeza closed this Sep 9, 2015
@joserprieto
Copy link

Probad el SmartGit; va bastante bien y suele solucionar esas cosillas de
forma fácil y rápida.
Al menos, en mi experiencia.
El 9/9/2015 7:40 p. m., "Pedro M. Baeza" notifications@github.com
escribió:

Closed #204 #204.


Reply to this email directly or view it on GitHub
#204 (comment).

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

3 participants