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

api-kit lacks support for zksync and zkevm #729

Closed
clemsos opened this issue Mar 11, 2024 · 2 comments · Fixed by #730
Closed

api-kit lacks support for zksync and zkevm #729

clemsos opened this issue Mar 11, 2024 · 2 comments · Fixed by #730

Comments

@clemsos
Copy link

clemsos commented Mar 11, 2024

Description

When trying to send a tx to a safe on either Zksync or Polygon Zkevm, the lib states that the txServiceUrl is missing - even though one can find it on safe.global UI

zkevm

TypeError: There is no transaction service available for chainId 1101. Please set the txServiceUrl property to use a custom transaction service.
    at new SafeApiKit (/Users/clement/Dev/unlock/unlock/node_modules/@safe-global/api-kit/src/SafeApiKit.ts:57:15)

zksync

TypeError: There is no transaction service available for chainId 324. Please set the txServiceUrl property to use a custom transaction service.
    at new SafeApiKit (/Users/clement/Dev/unlock/unlock/node_modules/@safe-global/api-kit/src/SafeApiKit.ts:53:15)

Environment

  • Safe Core SDK version: 2.2.0
  • Environment: hardhat / node

Steps to reproduce

The script used is here https://github.com/unlock-protocol/unlock/blob/9ac6a3ed44527cb4638bf15a5da7548e7c255495/governance/scripts/multisig/addOwner.js

Note that the scripts works for ohter networks (such as arbitrum, avalanche, base, bsc, celo, gnosis, mainnet, optimism, polygon)

Expected result

tx should be submitted like for other supported networks

@dasanra
Copy link
Collaborator

dasanra commented Mar 11, 2024

@clemsos Thank you for the ticket. True, those two chains are not added in the default configuration. We will add them. Meanwhile you can use the txServiceUrl parameter to set the corresponding link to the service for those two cases.

@clemsos
Copy link
Author

clemsos commented Mar 12, 2024

great, thank you !

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 a pull request may close this issue.

2 participants