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

Release 7.3 #3

Closed

nbessi wants to merge 19 commits into OCA:master from nbessi:payment-slip-evolution-dunning-fees-support-nbi

Contributor

nbessi commented Jun 25, 2014

Change with release 7.3

Remove compatibility code for BVR/ESR only Multi BVR/ESR is supported
Add credit control integration with ESR
Add intergation with account bank reconciliation addons https://launchpad.net/banking-addons/bank-statement-reconcile-7.0 with module l10n_ch_payment_slip_base_transaction_id
Uniformize management of reference

nbessi and others added 19 commits

April 15, 2014 13:28


          [REFACTOR] l10n_ch_payment_slip_report

074619e

* Removing deprecated BVR report from invoice
* PEP8
* Replacing deprecated except_osv by except_orm
* _space become a static method
* DRY some code


          [ADD] hook to allows to override BVR/ERS report total amount

781e069


          [REFACTOR] following fix of bug lp:1308364

7d2583d


          [RM] dead code

15d26a1


          [RM] forgotten pdb

453a4ed


          [REFATOR] credit control payment slip to support crdit.control.line a…

539760b

…nd move line relation cardinality


          [FIX] cleaning tree of unused files

00385cd


          [IMP] file name to use datetime

be0788c


          [FIX] addons version

1d625de


          [FIX] addons description et version

d611fa1


          [IMP] Change log

9f74227


          [FIX] community conventions

1044cfc


          add a summary

73d2073


          TYPO

7d5edb9


          add missing contexts

8b5bb1a


          fix indent

1d63cb6


          remove import of orm imported twice

686fb30


          [PEP8] in multi_report_webkit_html.py

5efa56f


          newish style for self.pool

e0182e4

Contributor Author

nbessi commented Jun 25, 2014

original MP:
https://code.launchpad.net/~camptocamp/openerp-swiss-localization/payment-slip-evolution-dunning-fees-support-nbi/+merge/218601

To work on OCB depends on lp:~camptocamp/ocb-addons/fix-1308364-nbi
To work on official depends on https://code.launchpad.net/~camptocamp/openobject-addons/7.0-fix-1308364-nbi

Depends on https://code.launchpad.net/~camptocamp/account-financial-tools/add-manual-line-and-fees-nbi/+merge/218584 to support of credit control dunning fees

This PR is an evolution of l10n_ch.

It removes duplicated code used for backward compatibility on BVR/ESR report. Only Multi-BVR/ESR report will remain.

It fixes multi-BVR/ESR report hack due to following bug:
https://bugs.launchpad.net/openobject-addons/+bug/1308364

It refactors and corrects l10n_ch_credit_control_payment_slip_report and add support of fees.

Contributor Author

nbessi commented Jun 25, 2014

Approved by @yvaucher on LP

Member

guewen commented Jun 25, 2014

⚠️ do not merge before we switch to github otherwise we may diverge from the launchpad branch

Contributor Author

nbessi commented Jun 25, 2014

@guewen yes, I will be careful and be sure both branch will not diverge.
As it is a big PR there probably will be some time before having more review.

nbessi added this to the 7.3 milestone

Contributor Author

nbessi commented Jul 2, 2014

Deprecated by #22

nbessi closed this

nbessi pushed a commit that referenced this pull request


          Merge pull request #3 from davidZIVI/7.0

Fix PEP8

vrenaville pushed a commit that referenced this pull request


          Merge pull request #3 from leemannd/10.0-migrate-l10n_ch_dta

e627897

Fix tests

yvaucher referenced this pull request in yvaucher/l10n-switzerland


          Merge pull request #3 from leemannd/10.0-migrate-l10n_ch_dta

c006f68

Fix tests

vrenaville referenced this pull request in camptocamp/l10n-switzerland


          Merge pull request #3 from yterrettaz/10.01

68850ba

10.01

vrenaville referenced this pull request in camptocamp/l10n-switzerland


          Merge pull request #3 from vrenaville/fix_reconcile_esr

b4a2aba

[FIX] typo  in function call

davidwul added a commit to davidwul/l10n-switzerland that referenced this pull request


          # This is a combination of 4 commits.

3f930a3

# This is the 1st commit message:

85a93e8

# This is the commit message OCA#2:

use qr_iban validation methode

# This is the commit message OCA#3:

fix commit

# This is the commit message OCA#4:

remove debug info

i-vyshnevska referenced this pull request in camptocamp/l10n-switzerland


          [FIX] remove customer fields (#3)

0de4c45

TDu referenced this pull request in camptocamp/l10n-switzerland


          [FIX] remove customer fields (#3)

770a0ed

SimoneVagile referenced this pull request in SimoneVagile/l10n-switzerland


          # This is a combination of 4 commits.

38a0d14

# This is the 1st commit message:

85a93e8

# This is the commit message #2:

use qr_iban validation methode

# This is the commit message #3:

fix commit

# This is the commit message #4:

remove debug info

SimoneVagile referenced this pull request in SimoneVagile/l10n-switzerland


          # This is a combination of 4 commits.

f01231e

# This is the 1st commit message:

85a93e8

# This is the commit message #2:

use qr_iban validation methode

# This is the commit message #3:

fix commit

# This is the commit message #4:

remove debug info

dzungtran89 pushed a commit to dzungtran89/l10n-switzerland that referenced this pull request


          [FIX] remove customer fields (OCA#3)

e790204

SimoneVagile referenced this pull request in SimoneVagile/l10n-switzerland


          # This is a combination of 4 commits.

b1a927b

# This is the 1st commit message:

85a93e8

# This is the commit message #2:

use qr_iban validation methode

# This is the commit message #3:

fix commit

# This is the commit message #4:

remove debug info

SimoRubi pushed a commit to SimoRubi/l10n-switzerland that referenced this pull request


          # This is a combination of 4 commits.

94ae4bf

# This is the 1st commit message:

85a93e8

# This is the commit message OCA#2:

use qr_iban validation methode

# This is the commit message OCA#3:

fix commit

# This is the commit message OCA#4:

remove debug info

JuMiSanAr referenced this pull request in camptocamp/l10n-switzerland


          Merge pull request #3 from vrenaville/master

8ad7756

[FIx] remove unused egg

JuMiSanAr referenced this pull request in camptocamp/l10n-switzerland


          -a is deprecated and may return False even if the file exists

82ea02d

Fixes #3

yvaucher pushed a commit that referenced this pull request


          [FIX] remove customer fields (#3)

f39538a

dzungtran89 referenced this pull request in chucorp/l10n-switzerland


          [FIX] remove customer fields (#3)

b2f1af4

dzungtran89 pushed a commit to dzungtran89/l10n-switzerland that referenced this pull request


          [FIX] remove customer fields (OCA#3)

4d42ec1

SimoneVagile referenced this pull request in SimoneVagile/l10n-switzerland


          # This is a combination of 4 commits.

7569cda

# This is the 1st commit message:

85a93e8

# This is the commit message #2:

use qr_iban validation methode

# This is the commit message #3:

fix commit

# This is the commit message #4:

remove debug info

ajaniszewska-dev referenced this pull request in camptocamp/l10n-switzerland


          Merge pull request #3 from ajaniszewska-dev/fix_quickpac

dfe8fbd

fix build

kv1612 pushed a commit to kv1612/l10n-switzerland that referenced this pull request


          # This is a combination of 4 commits.

943dba1

# This is the 1st commit message:

85a93e8

# This is the commit message OCA#2:

use qr_iban validation methode

# This is the commit message OCA#3:

fix commit

# This is the commit message OCA#4:

remove debug info

hbrunn pushed a commit to hbrunn/l10n-switzerland that referenced this pull request


          # This is a combination of 4 commits.

fe43e68

# This is the 1st commit message:

85a93e8

# This is the commit message OCA#2:

use qr_iban validation methode

# This is the commit message OCA#3:

fix commit

# This is the commit message OCA#4:

remove debug info

hailangvn referenced this pull request in camptocamp/l10n-switzerland


          # This is a combination of 4 commits.

828f6de

# This is the 1st commit message:

85a93e8

# This is the commit message #2:

use qr_iban validation methode

# This is the commit message #3:

fix commit

# This is the commit message OCA#4:

remove debug info

michelerusti pushed a commit to michelerusti/l10n-switzerland that referenced this pull request


          # This is a combination of 4 commits.

# This is the 1st commit message:

85a93e8

# This is the commit message OCA#2:

use qr_iban validation methode

# This is the commit message OCA#3:

fix commit

# This is the commit message OCA#4:

remove debug info

Ricardoalso referenced this pull request in camptocamp/l10n-switzerland


          # This is a combination of 4 commits.

bb11c80

# This is the 1st commit message:

85a93e8

# This is the commit message #2:

use qr_iban validation methode

# This is the commit message #3:

fix commit

# This is the commit message OCA#4:

remove debug info

OCA-git-bot pushed a commit that referenced this pull request


          # This is a combination of 4 commits.

5abd66c

# This is the 1st commit message:

85a93e8

# This is the commit message #2:

use qr_iban validation methode

# This is the commit message #3:

fix commit

# This is the commit message #4:

remove debug info

BT-anieto pushed a commit to BT-anieto/l10n-switzerland that referenced this pull request


          # This is a combination of 4 commits.

55242b1

# This is the 1st commit message:

85a93e8

# This is the commit message OCA#2:

use qr_iban validation methode

# This is the commit message OCA#3:

fix commit

# This is the commit message OCA#4:

remove debug info

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet