Skip to content

Use blockchain-api for transaction submission#1003

Merged
ChewingGlass merged 19 commits intomainfrom
feat/tx-api
Jan 27, 2026
Merged

Use blockchain-api for transaction submission#1003
ChewingGlass merged 19 commits intomainfrom
feat/tx-api

Conversation

@ChewingGlass
Copy link
Copy Markdown
Contributor

No description provided.

ChewingGlass and others added 9 commits December 2, 2025 09:33
* Tweaks to walletStandard.js

* fix: skip migration submit when no transactions to migrate

* tweaks

* fix: add missing chains getter to wallet-standard implementation

- add chains getter to HeliumWallet class for wallet-standard compliance
- register wallet immediately for apps already initialized (fixes jup.ag)
- guard property definition to prevent errors on re-injection
- remove dead code (unused variables)

* refactor: improve browser webview code organization

walletStandard.js:
- remove dead code (isSolanaChain, #reconnected, unused variables)
- rename features constant to WALLET_FEATURES for clarity
- use SOLANA_CHAINS directly instead of chains alias

BrowserWebViewScreen.tsx:
- extract deserializeTransactionInputs helper to reduce duplication
- split onMessage into separate handlers (handleConnect, handleSignAndSend, etc)
- extract BrowserHeader and BrowserFooter as proper typed components
- add postMessage helper for cleaner webview communication
- use switch statement for message type routing

* tweaks
@ChewingGlass ChewingGlass requested a review from a team as a code owner January 22, 2026 22:44
@ChewingGlass ChewingGlass requested review from Perronef5, allenan and tyler-whitman and removed request for a team January 22, 2026 22:44
@ChewingGlass ChewingGlass merged commit 88b9f1f into main Jan 27, 2026
3 of 4 checks passed
@ChewingGlass ChewingGlass deleted the feat/tx-api branch January 27, 2026 19:25
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.

2 participants