Skip to content

feat(wallet): wire TransactionController into default initialization#8988

Closed
matthewwalsh0 wants to merge 1 commit into
chore/remove-deprecated-tx-optionsfrom
feat/wallet-wire-tc-simple
Closed

feat(wallet): wire TransactionController into default initialization#8988
matthewwalsh0 wants to merge 1 commit into
chore/remove-deprecated-tx-optionsfrom
feat/wallet-wire-tc-simple

Conversation

@matthewwalsh0
Copy link
Copy Markdown
Member

Summary

  • Wires TransactionController into the wallet package's default initialization
  • Uses TransactionControllerMessenger directly (no separate init messenger) — same pattern as ApprovalController and KeyringController
  • Delegates all AllowedActions and AllowedEvents from the root messenger to the controller messenger via parent.delegate
  • Hardcodes incomingTransactions: { isEnabled: () => false } as both extension and mobile share this value
  • Exposes TransactionControllerInstanceOptions (omits messenger, state, incomingTransactions) in instanceOptions.transactionController

@matthewwalsh0
Copy link
Copy Markdown
Member Author

Closing — work moved back to #8975

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