-
Notifications
You must be signed in to change notification settings - Fork 39
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
Deeplinks #2679
Comments
To use deeplinks, do as follow opening the links in a new window (either connect your wallet
to connect your wallet it should be noted that the success callback url should contain so sign a txlooking at your issue that does not fully fulfill all requests, as a prepared TX is still required (and therefore most likely an SDK) in order to be signed. But thats what exists already: const rawTx = await aeSdk.buildTx(transaction);
const query = new URLSearchParams({
transaction: rawTx,
networkId: 'ae_uat', // or your network id
broadcast: "true",
});
const url = `https://wallet.superhero.com/sign-transaction?${query.toString()}&x-success=<success-url>&x-cancel=<cancel-url>`; sign a message
|
Hijacking this thread because it's relevant: This is indeed a really important feature and it was quite easy to integrate. The "sign a message" deeplink was almost exactly what we needed, with one missing feature about the encoding of the message: #2688 |
There were also a deeplink to make a spend transaction (including custom payload), also to claim an invite |
Hello everybody, I know, easier said than properly done, but as multiple approaches exist how we could pull this off with the way SH wallet works, we can find the most ideal one and go for it.
Is your feature request related to a problem? Please describe.
The motivations for this are as follows:
Describe the solution you'd like
Describe alternatives you've considered
Going down the SDK-route is not practical for parnters which support 14 different wallets, adding the whole SDK just to perform a simple
sign message
and/ortransfer
really bloats the dependencies.Additional context
Integration of aeternity on bigger platforms would heavily benefit from that, as we are learning in specific, not-yet-announced cases right now.
The text was updated successfully, but these errors were encountered: