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

Berachain Bartio + Combo Testnet #662

Merged
merged 3 commits into from
Jul 11, 2024

Conversation

TheDivic
Copy link
Collaborator

📖 Context

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
    🚨 The Combo Testnet chain ID changed

Why are we doing this?

  • Berachain has a new testnet called Bartio they want integrated
  • Combo testnet changed their chain ID

What did we do?

  • Changed the chain ID for Combo and added Bartio
  • Updated the gas estimations package to a new version to support Sei gas limit (still broken unfortunately)
  • Added Sei testnet to networks that don't support state overrides.

How Has This Been Tested?

  • ❌ Combo Testnet fails with aa13: EntryPoint failed to create the smart account with the initCode provided, waiting for response from the SDK/SC team.
  • ✅ Successfully submitted an userOp on Bartio
  • ❌ Sei still fails because of wrong gas limits, but it works when I hardcode them all to 500k.

Currently fails with aa14: The initCode returned a different smart account address than expected because of a wrong factory contract deployment
…stnet chain id

Bartio works like a charm, new Combo testnet fails with aa13: EntryPoint failed to create the smart account with the initCode provided
@TheDivic TheDivic added the ⛓️ Chain Integration Label for all chain integration related tasks label Jul 11, 2024
@TheDivic TheDivic self-assigned this Jul 11, 2024
@TheDivic TheDivic merged commit c877216 into master Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⛓️ Chain Integration Label for all chain integration related tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants