[16.0][MIG] account_banking_pain_base #974
Conversation
|
This will need these cahnges: |
|
@wpichler You added also the modules banking sepa direct debit and account_banking_mandate. |
The restriction rule Banking Mandate multi-company, gives an error when editing a partner bank account from a company A when that account has created a mandate from another company B. So it is impossible to create a mandate for each company without disabling the rule. Solution: Uncheck the rule writte in Banking Mandate multi-company. But i'm not sure this is correct.
* Short headers * Bump version numbers * es translations
* Add creditor identifier field to report mandate. * Translate. * Split basic and sepa mandate * Use api.multi instead * Make format field required
Port almost all modules to v10 * Update to EPC Rulebook v9.2 that start to apply on 2016-11-20 (bug OCA#300)
…ovided Using same method as in upstream, mandate is filled on invoice creation if no one is provided. This way, we don't need to install account_banking_mandate_sale if we don't want to handle several mandates at sales level.
…ically on the payment line'
…date + Add valid_mandate field and fix onchange (cherry picked from commit 25d2e4d)
add tests and constrains
* Fix tests due to upstream change Odoo has added a constraint for avoiding a company currency change if there are move lines, making these tests to fail, as the currency is changed to EUR. With this commit, we create a new company with EUR currency for avoiding the problem. This commit also changes account_banking_mandate for not duplicating mandate number, as it was detected during the test creation. Similar to 1f8e345 * Avoid errors in tests when run with other modules * More adaptations to make tests to work properly All these problems comes from using demo data.
Currently translated at 100,0% (82 of 82 strings) Translation: bank-payment-11.0/bank-payment-11.0-account_banking_mandate Translate-URL: https://translation.odoo-community.org/projects/bank-payment-11-0/bank-payment-11-0-account_banking_mandate/es/
Currently translated at 68,3% (56 of 82 strings) Translation: bank-payment-11.0/bank-payment-11.0-account_banking_mandate Translate-URL: https://translation.odoo-community.org/projects/bank-payment-11-0/bank-payment-11-0-account_banking_mandate/ca/
So multi-company environment won't work correctly.
wpichler
force-pushed
the
16.0-mig-account_banking_pain_base
branch
2 times, most recently
from
6097ab3 to
182ce00
Compare
Thank you - that was exact what i was looking for - works like a charm |
pedrobaeza
changed the title
|
/ocabot migration account_banking_pain_base |
|
@OCA/banking-maintainers There are some PR depending on this and others, Could you review and merge the maximum as possible? |
|
[DEL] Removed modules which do not belong to this PR [IMP] Precommit updates |
[REF] account_banking_pain_base: Adapt module to native payment refactoring [ADD] Added AT to list of supported countries for SEPA. Fixes OCA#1002 [DEL] Removed modules which do not belong to this PR [IMP] Precommit updates
wpichler
force-pushed
the
16.0-mig-account_banking_pain_base
branch
from
182ce00 to
29b53b2
Compare
|
@wpichler module |
|
A commit squashing and to include this commits is needed: 15661a2 |
|
cc @wpichler Do you need help for this PR ? to push forward |
|
cc @wpichler Any update about this PR ? |
This comment was marked as off-topic.
This comment was marked as off-topic.
|
The commit history of this one is totally mangled, with more than 250 commits where there's actually less than a hundred, so I have started the commit history fetching from the scratch. As there's no significant changes on the module itself either, and the migration commit of this one is not correct, I'm totally discarding it in favor of #1087, so closing this one and merging the other for finally unblocking this. |
#964