Skip to content

[#470] Update ZAP_PLOTLINK to fixed USDC deployment#471

Merged
realproject7 merged 1 commit intomainfrom
task/470-usdc-zap-final-fix
Mar 23, 2026
Merged

[#470] Update ZAP_PLOTLINK to fixed USDC deployment#471
realproject7 merged 1 commit intomainfrom
task/470-usdc-zap-final-fix

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Updates ZAP_PLOTLINK constant to the new deployment 0xAe50C9444DA2Ac80B209dC8B416d1B4A7D3939B0 which fixes USDC zap minting
  • Contract fix details in plotlink-contracts PR #65

Test plan

  • Mainnet USDC mint tx confirmed: 0x417378c8...
  • Verify trading widget works with new contract on staging

Fixes #470

🤖 Generated with Claude Code

New address: 0xAe50C9444DA2Ac80B209dC8B416d1B4A7D3939B0
Fixes USDC zap mint that was reverting due to V4 Router encoding mismatch.
See plotlink-contracts PR #65 for root cause analysis.

Fixes #470

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

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

Verdict: APPROVE

Summary

This app-side change is the minimal follow-up needed to point the frontend at the fixed Zap deployment. I did not find correctness or scope issues in the constants update.

Findings

  • None.

Decision

Approve. The change is limited to the production Zap address swap and correctly tracks the contract redeploy in plotlink-contracts#65.

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

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

T2b APPROVE

Clean, minimal constant update.

  • New address 0xAe50C9444DA2Ac80B209dC8B416d1B4A7D3939B0 matches redeployed contract from PR #65
  • EIP-55 checksum valid, type cast preserved
  • Mainnet tx confirms contract works

Recommendation: confirm staging widget verification (test plan item 2) before/after merge.

@realproject7 realproject7 merged commit 167ecf3 into main Mar 23, 2026
3 checks passed
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.

[Bug] USDC zap mint reverts — study MintPad contract and fix

2 participants