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

add encointer #3

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

brenzi
Copy link

@brenzi brenzi commented Aug 27, 2024

after polkadot-api/polkadot-api#641 I wanted to do the exercise to add encointer.

This would actually be useful because no mobile wallet is currently supporting Encointer teleports. thanks to dapp browsers, this PR could add that functionality without integration and testing pains on the wallet side

I do need help, however:

  • how do I generate .papi/metadata/ksmEnc.scale ?
  • polkadot-api 1.0.2 should inculde the PR mentioned above, yet pnpm build complains: Missing "./chains/ksmcc3_encointer" specifier in "polkadot-api" package

Copy link

@voliva voliva left a comment

Choose a reason for hiding this comment

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

Encointer hasn't been released in polkadot-api - Following semantic versioning, 1.0.2 is a bunch of fixes for the 1.0 release.

It will be released for 1.1.0. Once it's there, then you can easily add it with the CLI with pnpm papi add ksmEnc -n ksmcc3_encointer

papi supports adding chains that are not in the well-known chains though, so at this moment you could do it with local chainSpec file or a WS url. Try this: pnpm papi add ksmEnc -w wss://kusama.api.encointer.org.

You will need the chainSpec file if you want to use smoldot though - you could still use a websocket provider too.

These are suggestions if they help get a better picture of how papi deals with different chains. For this repo, I would wait until 1.1.0 is released before merging in.

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.

2 participants