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

xdc + zksync #1114

Merged
merged 1 commit into from
Dec 9, 2022
Merged

xdc + zksync #1114

merged 1 commit into from
Dec 9, 2022

Conversation

aznrayizzle
Copy link
Contributor

add xdc + zkSync tutorials

]
```

XDC has changed their wallet address [prefix] (https://medium.com/xinfin/xinfin-releases-new-address-prefix-starting-with-xdc-75750a444f77) to start with "xdc" instead of "0x".
Copy link
Member

Choose a reason for hiding this comment

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

think you need to remove the space between [prefix] and the link

- Connect to an ethereum network
- Use the `evmconnect` blockchain connector
- Use an remote RPC node. This will create a signer locally, so that our signing key never leaves the development machine.
- See the list of providers for zkSync [docs](https://v2-docs.zksync.io/dev/fundamentals/testnet.html). For this tutorial we will use https://zksync2-testnet.zksync.dev
Copy link
Member

Choose a reason for hiding this comment

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

Should we change the styling of the https://zksync2-testnet.zksync.dev link to make it appear different from non-link text.

@codecov-commenter
Copy link

Codecov Report

Merging #1114 (8535e78) into main (8565dda) will increase coverage by 0.04%.
The diff coverage is n/a.

❗ Current head 8535e78 differs from pull request most recent head 0efc587. Consider uploading reports for the commit 0efc587 to get more accurate results

@@            Coverage Diff             @@
##             main    #1114      +/-   ##
==========================================
+ Coverage   99.93%   99.97%   +0.04%     
==========================================
  Files         302      302              
  Lines       19426    19426              
==========================================
+ Hits        19414    19422       +8     
+ Misses          9        4       -5     
+ Partials        3        0       -3     
Impacted Files Coverage Δ
internal/apiserver/server.go 100.00% <0.00%> (+0.91%) ⬆️
internal/events/webhooks/webhooks.go 100.00% <0.00%> (+2.76%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Signed-off-by: aznrayizzle <ray.chen@kaleido.io>
Copy link
Member

@shorsher shorsher left a comment

Choose a reason for hiding this comment

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

Thanks @aznrayizzle! Looks great.

@shorsher shorsher merged commit 991af52 into hyperledger:main Dec 9, 2022
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.

None yet

3 participants