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

[8.0] Porting l10n_it_corrispettivi #82

Closed
wants to merge 16 commits into from
Closed

[8.0] Porting l10n_it_corrispettivi #82

wants to merge 16 commits into from

Conversation

shakin89
Copy link

This is my first attempt to port a module to v 8.0

@coveralls
Copy link

Coverage Status

Coverage decreased (-6.81%) when pulling fe5cfe9 on shakin89:8.0 into 7e8ae7c on OCA:8.0.

@coveralls
Copy link

Coverage Status

Coverage decreased (-6.81%) to 58.96% when pulling b0bdad1 on shakin89:8.0 into 7e8ae7c on OCA:8.0.

@eLBati
Copy link
Member

eLBati commented Jan 29, 2015

Thanks @shakin89
I see you started from l10n_it_corrispettivi version contained in 8.0 branch, but it is outdated. You should start porting from 7.0 branch.
See the diff between 7.0 version and your version:
https://gist.github.com/eLBati/d0b190881f72b356f2a2
As you can see, you removed some improvements made on 7.0 which must be kept (like from . import account)

So, please copy the 7.0 version of the module to a new branch and apply your commits on it.
Thanks

@shakin89
Copy link
Author

Ok, i've ported the improvement made in 7.0 version, i've compared all the files and aligned the 7.0 version to this one.

@coveralls
Copy link

Coverage Status

Coverage decreased (-6.81%) to 58.96% when pulling 4a92401 on shakin89:8.0 into 7e8ae7c on OCA:8.0.

@coveralls
Copy link

Coverage Status

Coverage decreased (-6.81%) to 58.96% when pulling 76fec06 on shakin89:8.0 into 7e8ae7c on OCA:8.0.

@eLBati
Copy link
Member

eLBati commented Jun 29, 2015

This gives conflicts and should be rebased

eLBati pushed a commit to eLBati/l10n-italy that referenced this pull request Jun 3, 2016
@eLBati
Copy link
Member

eLBati commented Jun 3, 2016

@shakin89 chiuderei questa PR per continuare il tuo lavoro sulla #230 e sistemare le ultime cose per fare il merge.
Fammi sapere se OK per te

@eLBati eLBati closed this Jun 3, 2016
SimoRubi pushed a commit to SimoRubi/l10n-italy that referenced this pull request May 22, 2018
SimoRubi pushed a commit to SimoRubi/l10n-italy that referenced this pull request Sep 3, 2018
SimoRubi pushed a commit to SimoRubi/l10n-italy that referenced this pull request Sep 3, 2018
SimoRubi pushed a commit to SimoRubi/l10n-italy that referenced this pull request Sep 5, 2018
SimoRubi pushed a commit to SimoRubi/l10n-italy that referenced this pull request Sep 18, 2018
monen17 pushed a commit to monen17/l10n-italy that referenced this pull request Jan 18, 2019
monen17 pushed a commit to monen17/l10n-italy that referenced this pull request Jan 19, 2019
monen17 pushed a commit to monen17/l10n-italy that referenced this pull request Jan 26, 2019
SimoRubi pushed a commit to SimoRubi/l10n-italy that referenced this pull request Aug 7, 2019
SimoRubi pushed a commit to SimoRubi/l10n-italy that referenced this pull request Aug 7, 2019
tafaRU pushed a commit to tafaRU/l10n-italy that referenced this pull request Nov 12, 2021
tafaRU pushed a commit to tafaRU/l10n-italy that referenced this pull request Mar 23, 2022
tafaRU pushed a commit to tafaRU/l10n-italy that referenced this pull request Jul 14, 2022
tafaRU pushed a commit to tafaRU/l10n-italy that referenced this pull request Jul 14, 2022
eLBati pushed a commit to eLBati/l10n-italy that referenced this pull request Oct 10, 2022
eLBati pushed a commit to eLBati/account-invoicing that referenced this pull request Oct 21, 2022
OCA-git-bot pushed a commit to eLBati/account-invoicing that referenced this pull request Feb 2, 2023
Ricardoalso pushed a commit to camptocamp/account-invoicing that referenced this pull request Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants