OCA / l10n-switzerland

Odoo Swiss localization
GNU Affero General Public License v3.0
53 stars 163 forks source link

[13.0][MIG] l10n_ch_pain_base #541

Closed pegonzalezsredcor closed 2 years ago

pegonzalezsredcor commented 4 years ago

Module installable in v13 Adapted account.move.line.move_id Adapted _is_isr_ref instead of is_isr_reference Waiting on the following to do a final functional revision:

EDIT: @yvaucher list dependencies

yvaucher commented 4 years ago

restarting travis jobs now that bank-payment dependency is merged

pedrobaeza commented 4 years ago

@yvaucher what is the status of this? These modules are installed in the OCA instance and we need it for the DB migration

yvaucher commented 4 years ago

@pedrobaeza did not have the chance to dive into this module specificaly, we have been using the account_sepa enterprise modules lately. So I would be ok to do some optimistic merge on this. However, AFAIK those modules have worked for standard case but they might not take into account the last changes in l10n_ch. Which is currently still a bit messy. And this state of the official module has impact on most modules in this localization.

pedrobaeza commented 4 years ago

Well, we can put these PRs in the OCA instance, but I don't feel comfortable enough with the result for this part, but this is out of the scope of the bid for migrating the DB, as they were added later and they are country specific. It would be good that you check them when possible.

i-vyshnevska commented 3 years ago

@pegonzalezsredcor can you rebase PR, dependent PR's had been merged

yvaucher commented 3 years ago

@pegonzalezs can you rebase to make sure the CI passes now that all dependencies are merged?

github-actions[bot] commented 2 years ago

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.