Skip to content

[19.0][MIG] account_statement_import_online_plaid - #951

Open
dnplkndll wants to merge 9 commits into
OCA:19.0from
ledoent:19.0-mig-plaid
Open

[19.0][MIG] account_statement_import_online_plaid#951
dnplkndll wants to merge 9 commits into
OCA:19.0from
ledoent:19.0-mig-plaid

Conversation

@dnplkndll

@dnplkndll dnplkndll commented May 12, 2026

Copy link
Copy Markdown

Migrates account_statement_import_online_plaid from 18.0 to 19.0.

Migration changes ([19.0][MIG] commit)

  • Bump manifest to 19.0.1.0.0.
  • Modernise translations per pylint_odoo 19.0 (W8161 prefer-env-translation
    • W8301 translation-not-lazy): replace from odoo import _ and
      _("... %s") % var with self.env._("... %s", var).

No other 19.0 API breakage in this module — no _sql_constraints, auto_join,
groups_id, or odoo.osv.expression usage. The _obtain_statement_data /
_get_transactions contracts with account_statement_import_online are
unchanged.

Tests

12 tests pass locally on Odoo 19.0 (using mocked plaid SDK — no live API
required):

$ docker compose run --rm odoo-19 odoo -d odoo19_base \
    -i account_statement_import_online_plaid \
    --stop-after-init --no-http --test-enable
Module account_statement_import_online_plaid loaded in 1.11s
0 failed, 0 error(s) of 12 tests

CI on the branch is also green:

  • pre-commit ✅
  • ruff / ruff format ✅
  • pylint_odoo ✅
  • prettier (XML) ✅

A follow-up PR will add account-id filtering (preventing cross-account
transaction duplicates when a Plaid item contains multiple bank accounts —
we've been carrying that fix downstream on 18.0 for months) — out of scope
for this PR which is the straight migration.

/ocabot migration account_statement_import_online_plaid

@OCA-git-bot OCA-git-bot added series:19.0 mod:account_statement_import_online_plaid Module account_statement_import_online_plaid labels May 12, 2026
@dnplkndll dnplkndll changed the title [19.0][MIG] plaid [19.0][MIG] account_statement_import_online_plaid May 12, 2026
Adasat Torres de Leon and others added 2 commits May 19, 2026 10:08
[FIX]account_statement_import_provider_plaid: Changes in the amount signs

[FIX]account_statement_import_online_plaid: A verification has been added for the language to be used.

[IMP]account_statement_import_online_plaid: The maximum number of records per request has been increased.

[FIX] account_statement_import_online_plaid: Documentation
@kobros-tech

Copy link
Copy Markdown

Hi 👋

@dnplkndll
dnplkndll force-pushed the 19.0-mig-plaid branch 2 times, most recently from e9caa04 to 155747d Compare May 29, 2026 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:account_statement_import_online_plaid Module account_statement_import_online_plaid series:19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants