Skip to content

Fix crypto profile notification i18n#58

Merged
Pigbibi merged 1 commit into
mainfrom
codex/fix-crypto-i18n-namespace
Jun 4, 2026
Merged

Fix crypto profile notification i18n#58
Pigbibi merged 1 commit into
mainfrom
codex/fix-crypto-i18n-namespace

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • localize both crypto_live_pool_rotation and legacy crypto_leader_rotation strategy names in notifications
  • update crypto-strategies dependency pin to v0.4.8
  • add coverage for legacy profile localization and settings resolution

Validation

  • python -m ruff check notify_i18n_support.py tests/test_notify_i18n.py tests/test_runtime_config_support.py
  • PYTHONPATH=.:../QuantPlatformKit/src:../CryptoStrategies/src python -m unittest tests.test_runtime_config_support
  • PYTHONPATH=.:../QuantPlatformKit/src:../CryptoStrategies/src python - <<'PY' ... legacy/new notify localization assertions ... PY

Note

  • tests.test_notify_i18n was not runnable on this VPS because the local environment is missing numpy; the failure occurs while importing quant_platform_kit.binance.market_data before test execution.

@Pigbibi Pigbibi merged commit 3c49163 into main Jun 4, 2026
1 of 2 checks passed
@Pigbibi Pigbibi deleted the codex/fix-crypto-i18n-namespace branch June 4, 2026 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant