Skip to content

fix(komodo_defi_types): support TRON explorer URLs#338

Merged
CharlVS merged 2 commits intodevfrom
fix/tron-explorer-urls
Mar 24, 2026
Merged

fix(komodo_defi_types): support TRON explorer URLs#338
CharlVS merged 2 commits intodevfrom
fix/tron-explorer-urls

Conversation

@CharlVS
Copy link
Copy Markdown
Collaborator

@CharlVS CharlVS commented Mar 24, 2026

Summary

  • normalize explorer URL parsing so TRON hash-routed explorer configs resolve correctly
  • keep TRON/TRC20 marked as not exposing KDF-backed tx history in protocol metadata
  • align the TRON protocol regression test with the current explorer config shape
  • include the pending SDK build config and test cleanup commit already on this branch

Verification

  • dart analyze lib/src/coin_classes/protocol_class.dart lib/src/protocols/base/explorer_url_pattern.dart test/tron_protocol_test.dart
  • dart analyze test/activation/tron_activation_strategy_test.dart test/transaction_history/transaction_history_strategies_test.dart

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@github-actions
Copy link
Copy Markdown
Contributor

Visit the preview URL for this PR (updated for commit 1d3b377):

https://komodo-playground--pr338-fix-tron-explorer-ur-fd4n4ama.web.app

(expires Tue, 31 Mar 2026 16:54:24 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 2bfedd77fdea45b25ba7c784416e81f177aa5c47

@CharlVS CharlVS merged commit ec8e714 into dev Mar 24, 2026
2 of 5 checks passed
@CharlVS CharlVS deleted the fix/tron-explorer-urls branch March 24, 2026 17:58
CharlVS added a commit that referenced this pull request Apr 1, 2026
* fix(komodo_defi_types): support TRON explorer URLs

* chore(sdk): commit pending SDK updates
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