Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not supported chain id 2345 #29840

Open
1 of 9 tasks
eigmax opened this issue Jan 22, 2025 · 2 comments
Open
1 of 9 tasks

Not supported chain id 2345 #29840

eigmax opened this issue Jan 22, 2025 · 2 comments

Comments

@eigmax
Copy link

eigmax commented Jan 22, 2025

What is this about?

When we use "https://gas.api.cx.metamask.io/networks/2345/suggestedGasFees" for GOAT Network to get the estimated gas, it responds:

{
 "statusCode": 400,
 "message": "'2345' is not a supported chain id."
}

We have listed GOAT on chainlist at https://chainlist.org/chain/2345.

Scenario

When the user add GOAT Network into their metamask, but Metamask can not get the correct gas for their transaction.

Design

No response

Technical Details

We dived into the issue and found that https://gas.api.cx.metamask.io/networks/2345/suggestedGasFees doesn't work. and from this comment (#25655 (comment)), it says suggestedGasFees doesn't use the chain's eth_estimateGas.

We need to use our own eth_estimateGas because GOAT Network is a Bitcoin L2, and the gas token is wrapped BTC.
GOAT Network:

Threat Modeling Framework

No response

Acceptance Criteria

No response

Stakeholder review needed before the work gets merged

  • Engineering (needed in most cases)
  • Design
  • Product
  • QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
  • Security
  • Legal
  • Marketing
  • Management (please specify)
  • Other (please specify)

References

No response

@dbrans
Copy link
Contributor

dbrans commented Jan 22, 2025

Duplicate of #29719

@dbrans dbrans marked this as a duplicate of #29719 Jan 22, 2025
@eigmax
Copy link
Author

eigmax commented Jan 22, 2025

Thanks for redirecting, @dbrans

Is there any update for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants