Skip to content

[16.0][MIG] currency_rate_update_wise#223

Open
klodr wants to merge 1 commit into
OCA:16.0from
klodr:16.0-mig-currency_rate_update_wise
Open

[16.0][MIG] currency_rate_update_wise#223
klodr wants to merge 1 commit into
OCA:16.0from
klodr:16.0-mig-currency_rate_update_wise

Conversation

@klodr

@klodr klodr commented Jun 20, 2026

Copy link
Copy Markdown

currency_rate_update_wise → 16.0

Forward-port of the Wise.com exchange-rate provider from 14.0 to 16.0.
Framework migration only, no functional change.

Notes for reviewers

  • Hermetic tests: the Wise HTTP layer (_wise_provider_retrieve) is mocked —
    no network access needed in CI.
  • Carries a small, non-blocking helper that logs a warning when the configured
    API token still uses the legacy UUID format (Wise UUID → JWT migration).

@OCA-git-bot OCA-git-bot added series:16.0 mod:currency_rate_update_wise Module currency_rate_update_wise labels Jun 20, 2026
@klodr klodr force-pushed the 16.0-mig-currency_rate_update_wise branch 2 times, most recently from 60ade8f to 6824c07 Compare June 20, 2026 06:51
Forward-port of the renamed and hardened Wise exchange-rate provider introduced
on 14.0 (rename from currency_rate_update_transferwise + security fixes; see the
14.0 history for the rationale and the OpenUpgrade rename). Framework migration
only, no functional change.

Signed-off-by: Claude Perrin <klodr@users.noreply.github.com>
@klodr klodr force-pushed the 16.0-mig-currency_rate_update_wise branch from 6824c07 to 95e30f2 Compare June 20, 2026 08:11
@klodr

klodr commented Jun 20, 2026

Copy link
Copy Markdown
Author

A note on the red CI, for reviewers:

  • pre-commit fails because the setuptools-odoo 3.1.8 hooks import the removed pkg_resources; this is fixed by the prerequisite bump [16.0] pre-commit: bump setuptools-odoo to 3.3.2 #227.
  • test with OCB / test with Odoo fail solely on the pre-existing currency_rate_update_xe module — its test_cron / test_wizard make real network calls to xe.com (which no longer returns usable data), asserting 0 != 1. This is unrelated to this PR.

currency_rate_update_wise itself installs cleanly and all 10 of its tests pass — the Wise HTTP layer is mocked, so no network access is needed in CI. The xe.com test issue is addressed for 16.0 in #216 (and #218 for 17.0, #220 for 14.0).

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

Labels

mod:currency_rate_update_wise Module currency_rate_update_wise series:16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants