Skip to content

Fix amoy rpc and gas estimation - #28

Closed
ankitkr104 wants to merge 2 commits into
StabilityNexus:mainfrom
ankitkr104:fix-amoy-rpc-and-gas-estimation
Closed

ankitkr104 wants to merge 2 commits into
StabilityNexus:mainfrom
ankitkr104:fix-amoy-rpc-and-gas-estimation

Conversation

@ankitkr104

@ankitkr104 ankitkr104 commented Aug 13, 2026 •

Copy link
Copy Markdown
Contributor

Addressed Issues:

Fixes #(TODO:issue number)

Screenshots/Recordings:

TODO: If applicable, add screenshots or recordings that demonstrate the interface before and after the changes.

Additional Notes:

AI Usage Disclosure:

We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact. AI slop is strongly discouraged and may lead to banning and blocking. Do not spam our repos with AI slop.

Check one of the checkboxes below:

  • This PR does not contain AI-generated code at all.
  • This PR contains AI-generated code. I have read the AI Usage Policy and this PR complies with this policy. I have tested the code locally and I am responsible for it.

I have used the following AI models and tools: TODO

Checklist

  • My PR addresses a single issue, fixes a single bug or makes a single improvement.
  • My code follows the project's code style and conventions
  • If applicable, I have made corresponding changes or additions to the documentation
  • If applicable, I have made corresponding changes or additions to tests
  • My changes generate no new warnings or errors
  • I have joined the Discord server and I will share a link to this PR with the project maintainers there
  • I have read the Contribution Guidelines
  • Once I submit my PR, CodeRabbit AI will automatically review it and I will address CodeRabbit's comments.
  • I have filled this PR template completely and carefully, and I understand that my PR may be closed without review otherwise.

Summary by CodeRabbit

  • New Features

    • Added wallet connection requirements for minting and redeeming assets.
    • Transactions now target the connected wallet address and use updated Amoy network fee settings.
    • Added clearer transaction submission and confirmation error messages.
  • Bug Fixes

    • Updated network connectivity and contract configuration for more reliable Amoy transactions.
    • Improved mint and redeem transaction handling by including recipient addresses.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026 •

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 88819d32-79fe-4015-90d3-7b262b4e2772

📥 Commits

Reviewing files that changed from the base of the PR and between 717285d and 40edc84.

📒 Files selected for processing (3)
  • app/deployments/[id]/DeploymentDetailClient.tsx
  • config/wagmi.ts
  • lib/abi/Tectonic.ts

Walkthrough

The PR updates Tectonic ABI receiver arguments, configures Wagmi HTTP transports, and changes deployment mint and redeem submissions to require a connected wallet and use Amoy fee overrides.

Changes

Wallet Transaction Flow

Layer / File(s) Summary
Tectonic transaction inputs
lib/abi/Tectonic.ts
Mint methods now accept a receiver address. Redeem methods now accept an amount and receiver address.
Wagmi transport configuration
config/wagmi.ts
HTTP transports now cover supported chains, Polygon Amoy, and local Hardhat. The fallback WalletConnect project ID changed.
Deployment wallet submission
app/deployments/[id]/DeploymentDetailClient.tsx
Submission now requires a connected wallet, passes the wallet address to contract methods, applies Amoy fee settings, and displays write or receipt errors.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant DeploymentDetailClient
  participant Wagmi
  participant TectonicContract
  User->>DeploymentDetailClient: Submit mint or redeem
  DeploymentDetailClient->>Wagmi: Read connected wallet address
  DeploymentDetailClient->>TectonicContract: Call method with receiver and Amoy fee overrides
  TectonicContract-->>DeploymentDetailClient: Return write or receipt error
  DeploymentDetailClient-->>User: Display transaction error
Loading

Possibly related PRs

Suggested labels: Typescript Lang

Poem

A rabbit connects its wallet bright,
Then mints and redeems with fees set right.
Receiver addresses guide each call,
Amoy transports carry them all.
Errors appear when transactions fall.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ankitkr104 ankitkr104 closed this Aug 13, 2026
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