Skip to content

feat: add example Tempo batch tx#427

Merged
maxime-oe merged 2 commits intomainfrom
feat/add-tempo-batch-tx
Mar 6, 2026
Merged

feat: add example Tempo batch tx#427
maxime-oe merged 2 commits intomainfrom
feat/add-tempo-batch-tx

Conversation

@maxime-oe
Copy link
Contributor

@maxime-oe maxime-oe commented Feb 25, 2026

Note

Medium Risk
Introduces new transaction-sending UI that can trigger real on-chain ERC20 transfers using a custom tx type and fee token fields. Risk is mainly accidental execution/misconfiguration rather than deep system impact.

Overview
Adds a new “Tempo Transactions” test component that renders inputs for chainId, ERC20 token address, and fee token, plus a button to submit a Tempo type: 0x76 batched eth_sendTransaction with two hardcoded ERC20 transfer calls.

Wires this component into src/index.js so it appears in the app UI and enables/disables it based on the existing globalConnectionChange / disableAndClear events, displaying the resulting tx hash (or error) inline.

Written by Cursor Bugbot for commit 5470e68. This will update automatically on new commits. Configure here.

@maxime-oe maxime-oe force-pushed the feat/add-tempo-batch-tx branch from f996074 to 7f6d5ac Compare March 5, 2026 16:42
@maxime-oe maxime-oe marked this pull request as ready for review March 5, 2026 16:42
@maxime-oe maxime-oe force-pushed the feat/add-tempo-batch-tx branch 2 times, most recently from 03f57c8 to 092ff6c Compare March 5, 2026 16:51
@maxime-oe maxime-oe force-pushed the feat/add-tempo-batch-tx branch from 092ff6c to b3d21d9 Compare March 5, 2026 17:13
Copy link
Member

@seaona seaona left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added a couple of small suggestions. Let me know if they make sense!

// Tempo Transactions

// ERC20 TST token created and owned by shared account 0x13b7e6EBcd40777099E4c45d407745aB2de1D1F8
const erc20TokenAddress = '0x86fA047df5b69df0CBD6dF566F1468756dCF339D';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for adding a bit more flexibility in the testing side, could we add an input element in the front end, so we can pass our custom ERC20 token address if we wish?
This address could be taken as the default value, if none is provided.
Wdyt?

// ERC20 TST token created and owned by shared account 0x13b7e6EBcd40777099E4c45d407745aB2de1D1F8
const erc20TokenAddress = '0x86fA047df5b69df0CBD6dF566F1468756dCF339D';
const chainId = '0x89'; // Forcing to Polygon PoS for now since EIP7702 not available on Tempo
const feeToken = '0x3c499c542cef5e3811e1192ce70d8cc03d5c3359'; // USDC Coin (PoS) for Polygon Mainnet testing
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same to these, maybe if we can choose that from the UI as well, we won't need additional changes later (ie changing the chaind)

@maxime-oe
Copy link
Contributor Author

@seaona Ok will do a form and use the current values as default to maintain the aspect "one click to test" possible.

@maxime-oe maxime-oe force-pushed the feat/add-tempo-batch-tx branch from b3d21d9 to 7ad8413 Compare March 6, 2026 14:15
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

@maxime-oe maxime-oe force-pushed the feat/add-tempo-batch-tx branch from 7ad8413 to feeaf74 Compare March 6, 2026 14:19
@maxime-oe
Copy link
Contributor Author

@seaona comments addressed ;)
Simple input texts pre-filled with what were the default values.

@maxime-oe
Copy link
Contributor Author

image

@maxime-oe maxime-oe merged commit bae190b into main Mar 6, 2026
9 checks passed
@maxime-oe maxime-oe deleted the feat/add-tempo-batch-tx branch March 6, 2026 15: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