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.

Sign up for GitHub

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

Jump to bottom

Fiscal Code base on franksl branch #62

Merged
merged 23 commits into from
Dec 23, 2014
Merged

Fiscal Code base on franksl branch #62

merged 23 commits into from
Dec 23, 2014

Conversation

scigghia
Copy link
Contributor

@scigghia scigghia commented Dec 2, 2014

No description provided.

@eLBati
Copy link
Member

eLBati commented Dec 2, 2014

Based on #35

@scigghia
Copy link
Contributor Author

scigghia commented Dec 2, 2014

that's all folks

@eLBati
Copy link
Member

eLBati commented Dec 2, 2014

Thanks @scigghia
Here scigghia#1 is how I propose to fix travis

@tafaRU tafaRU added the needs fixing Has conflicts or is failing mandatory CI checks label Dec 6, 2014
@coveralls
Copy link

Coverage Status

Coverage decreased (-34.04%) when pulling 42ed942 on scigghia:8.0 into 0e6f505 on OCA:8.0.

@eLBati
Copy link
Member

eLBati commented Dec 15, 2014

LGTM 👍

@tafaRU tafaRU added needs review and removed needs fixing Has conflicts or is failing mandatory CI checks labels Dec 15, 2014
@eLBati eLBati mentioned this pull request Dec 19, 2014
Merged
LeartS
LeartS reviewed Dec 19, 2014
View reviewed changes
l10n_it_fiscalcode/model/res_partner.py
help="If checked the C.F. is referred to a Individual Person")

_defaults = {
'individual': False,
Copy link
Contributor

Choose a reason for hiding this comment

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

The new way™ of specifying defaults it to pass them as parameters to the field constructor. 1 2

@eLBati eLBati added needs fixing Has conflicts or is failing mandatory CI checks and removed needs review labels Dec 22, 2014
@coveralls
Copy link

Coverage Status

Coverage decreased (-34.22%) when pulling c09f69d on scigghia:8.0 into 0e6f505 on OCA:8.0.

@coveralls
Copy link

Coverage Status

Coverage decreased (-34.22%) when pulling 0b5a8af on scigghia:8.0 into 0e6f505 on OCA:8.0.

@scigghia
Copy link
Contributor Author

👍

1 similar comment
@giudalo
Copy link

giudalo commented Dec 23, 2014

👍

@lepistone
Copy link
Member

Two approvals, and my comments on the other PR were addressed. Thanks!

lepistone added a commit that referenced this pull request Dec 23, 2014
@lepistone
Merge pull request #62 from scigghia/8.0
4ec76fa 
Fiscal Code based on franksl branch
@lepistone lepistone merged commit 4ec76fa into OCA:8.0 Dec 23, 2014
@yvaucher yvaucher removed the needs fixing Has conflicts or is failing mandatory CI checks label Dec 23, 2014
eLBati added a commit to archetipo/l10n-italy that referenced this pull request Jun 10, 2015
@eLBati
Merge pull request OCA#62 from archetipo/fix_filename_case_sensitive
ad881a0 
Fix filename case sensitive
OpenCode pushed a commit to OpenCode/l10n-italy that referenced this pull request May 9, 2017
@gurneyalex
add failing test for OCA#62
75f724d
vincenzoterzulli pushed a commit to ElvenStudio/l10n-italy that referenced this pull request Jul 11, 2022
@d-stragapede
Merge pull request OCA#62 from elvenstudiotmp/OCA8.0_fatturapa_pec
0eb8dd3 
[8.0][FIX] l10n_it_fatturapa_pec
patrickt-oforce pushed a commit to patrickt-oforce/l10n-italy that referenced this pull request Jul 11, 2023
@matteoopenf
Merge pull request OCA#62 from openforceit/12.0-5657-fix_withholding_tax
2808f0b 
12.0 5657 fix withholding tax
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@scigghia @eLBati @coveralls @giudalo @lepistone @LeartS @tafaRU @yvaucher