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

ISC - Exchange Guide for ShimmerEVM #1502

Closed
wants to merge 5 commits into from
Closed

Conversation

lucas-tortora
Copy link
Collaborator

Description of change

first draft, exchange guide for ShimmerEVM

DO NOT MERGE THIS UNTIL IT'S BEEN REVIEWED BY A SUBJECT EXPERT

Links to any relevant issues

fixes iotaledger/devx#319

Type of change

  • Documentation

Change checklist

  • I have followed the contribution guidelines for this project
  • I have performed a self-review of my own changes
  • I have made sure that added/changed links still work
  • I have commented my code, particularly in hard-to-understand areas

@lucas-tortora lucas-tortora marked this pull request as ready for review February 29, 2024 16:12
@lucas-tortora lucas-tortora marked this pull request as draft February 29, 2024 16:12
Copy link
Contributor

github-actions bot commented Feb 29, 2024

This pull request has been deployed to Vercel.

Latest commit: d97ae4c

✅ Preview: https://iota-wiki-9wsx0rnpl-iota1.vercel.app/wasp-evm/how-tos/integrate-your-exchange/

Comment on lines 14 to 16
URL (https://json-rpc.evm.shimmer.network) and ChainId (<ChainId url='https://json-rpc.evm.shimmer.network'/>), and run
our [Hornet](/hornet/welcome/) node with the [Wasp](/wasp/running-a-node/) INX-plugin.

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
URL (https://json-rpc.evm.shimmer.network) and ChainId (<ChainId url='https://json-rpc.evm.shimmer.network'/>), and run
our [Hornet](/hornet/welcome/) node with the [Wasp](/wasp/running-a-node/) INX-plugin.
URL (https://json-rpc.evm.shimmer.network) and ChainId (<ChainId url='https://json-rpc.evm.shimmer.network'/>), or set up your own [Hornet](/hornet/welcome/) node with the [Wasp](/wasp/running-a-node/) INX-plugin and get peered with the existing ShimmerEVM network.

If you use the public endpoint, you don't need to run your own node cluster right? So either connect to the public endpoint, or use your own cluster of Hornet and Wasp nodes. However, in the latter case, there is a procedure to be followed to actually get peered to the existing network of ShimmerEVM Wasp nodes right?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes, which isn't documented yet tho 🤔 So we should extent it once it is. SoonTM

Copy link
Contributor

@jorgemmsilva jorgemmsilva left a comment

Choose a reason for hiding this comment

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

very low quality.


If you already support EVM, then you probably are already familiar with what you'll need to integrate ShimmerEVM. As
with most networks, you should be able to use the same [JSON RPC API](https://wiki.iota.org/wasp-evm/reference/json-rpc-spec/) or clients, like [go-ethereum](https://geth.ethereum.org/), taking into account the
[compatibility and limitations](../getting-started/compatibility.md), use the ShimmerEVM RPC
Copy link
Contributor

Choose a reason for hiding this comment

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

this page linked is full of wrong/irrelevant info:
https://iota-wiki-9wsx0rnpl-iota1.vercel.app/wasp-evm/getting-started/compatibility/#properties-and-limitations

  • why do we talk about "on-ledger" requests on "EVM limitations"? that's not part of the EVM...
  • "Wrapped Calls to the JSON-RPC" ??? why does this even matter? it's just an implementation detail.
  • "Contract ID Source", "WASM Root Contract List", wtf...
  • "EVM state resides in a raw form to prevent the duplication of efforts undertaken by the ISC layer" - what?

that entire page should be scrapped.

please just keep:

  • No Enforced Block Time - use the "shimmerEVM" avg block time instead
  • The Magic Contract - use "Layer 1 tokens" ,instead of ISC tokens , remove "generally avail"
  • Gas Fees - rewrite and simply state that "the gas price is set by the chain"

@lucas-tortora lucas-tortora deleted the ISC/exchangeGuide branch March 25, 2024 20:26
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.

Create an exchange guide for IOTA EVM
4 participants