Uniswap / Uniswap/interface

[Bug] Data modification problem - onModification

Open
#7,946 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
TypeScript
Stars
5.5k
Forks
5.4k
PR merge metrics
No merged PRs in 30d

Description

📱 Interface Affected

Which application are you experiencing issues with?


🧩 App Version

  • Version (if known):
  • Production build
  • Development build

💻 System / Environment Info

Please provide details about your environment:

  • Browser (name + version): Chrome Version 139.0.7258.157
  • OS / Platform (e.g. iOS 17, Windows 11, Android 14): Windows 11
  • Device (e.g. iPhone 14 Pro, Pixel 7, MacBook Pro 2023): Asus vivobook
  • Wallet used (e.g. Uniswap Wallet, MetaMask, Rainbow): Hinkal Wallet, Avocado Wallet( any smart contract wallet)
  • Network (e.g. Ethereum Mainnet, Arbitrum, Base, etc.): Any network.

🔁 Steps to Reproduce

  1. Start a swap that requires an ERC-20 approval.
  2. Confirm the approval transaction in the wallet.
  3. Observe the approval step result and transactions UI/state.
  4. Proceed to the swap step (optional) and observe that swap tracking works as expected.

✅ Expected Behavior

What should have happened?
It should show successful transaction.


❌ Actual Behavior

What actually happened?
It showing that transaction was failed.


📸 Screenshots or Screen Recording

Please upload any relevant screenshots or recordings to help us understand the issue better.

Image

🧾 Additional Context

Any extra details? (e.g. logs, error messages, recent updates, beta flags enabled, etc.)
I go in the uniswap code and what I detect is that, since transaction data is changed from smart contract wallet, it runs interfaceUpdateTransactionInfo of slice.ts, but problem is that addTransaction is not called before. This is happening only for approval transaction step.


⚠️ Please redact or avoid sharing sensitive data such as private keys, seed phrases, or personally identifying info.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in slice.ts, focusing on interfaceUpdateTransactionInfo and the approval flow around addTransaction. Reproduce an ERC-20 approval swap with a smart contract wallet, then verify that the approval transaction is tracked and shown as successful while the swap tracking remains correct.

Written by the indexing model from the issue text.

Assessment

Tech stack
blockchain, typescript
Domain
blockchain, frontend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.