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

Support for Moonbeam addresses in Signer + Extension #693

Open
3 of 10 tasks
montekki opened this issue Sep 8, 2022 · 3 comments
Open
3 of 10 tasks

Support for Moonbeam addresses in Signer + Extension #693

montekki opened this issue Sep 8, 2022 · 3 comments
Labels
Enhancement Improves existing features or performance, making the SDK more robust and efficient. P3 - Low Non-essential improvements or minor fixes. Can be scheduled flexibly as time permits.

Comments

@montekki
Copy link

montekki commented Sep 8, 2022

  • I'm submitting a ...
  • Bug report
  • Feature request
  • Support request
  • Other
  • What is the current behavior and expected behavior?

With the start of development of support for Eth-like addresses in Signer in novasamatech/parity-signer#1268 we've found out that adding these arrds from Signer into the Extension with QR codes does not work. The format that we are using now is ethereum:ETH_KEY:GENESIS_HASH just like substrate:SS58_KEY:GENESIS_HASH for any other network

Full working support of Ethereum addresses and keys between Signer and Polkadot-js

  • What is the motivation for changing the behavior?

So people could use Signer on Moon* networks.

  • Please tell us about your environment:
  • Version: 0.44.1

  • Environment: Firefox

    • Node.js
    • Browser
    • Other (limited support for other environments)
  • Language:

    • JavaScript
    • TypeScript (include tsc --version)
    • Other
@polkadot-js-bot
Copy link

This issue has been open for 21 days with no activity and is not labelled as an enhancement. It will be closed in 7 days.

@jacogr
Copy link
Member

jacogr commented Sep 29, 2022

Most probably needs to move to the ui repo - QRs are parsed in the react-qr package ( as I recall).

EDIT: Indeed, there is only Substrate support (prefix being the substrate constant),

export function createAddressPayload (address: string, genesisHash: string): Uint8Array {
return encodeString(`${ADDRESS_PREFIX}:${address}:${genesisHash}`);
}

So would have some bubble-up effects in the display component as well where it would need additional flags (and then finally to the users of these components). Most probably needs some optional (important, needs backwards compat for existing users) flag to indicate anything that is non-Substrate prefixed.

@jacogr jacogr transferred this issue from polkadot-js/extension Nov 20, 2022
@jacogr jacogr added -size-m and removed -size-s labels Nov 20, 2022
@TarikGul TarikGul added Enhancement Improves existing features or performance, making the SDK more robust and efficient. P3 - Low Non-essential improvements or minor fixes. Can be scheduled flexibly as time permits. labels Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improves existing features or performance, making the SDK more robust and efficient. P3 - Low Non-essential improvements or minor fixes. Can be scheduled flexibly as time permits.
Projects
Development

No branches or pull requests

5 participants
@montekki @jacogr @TarikGul @polkadot-js-bot and others