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

First version CoinosProvider interface #1

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

louisinger
Copy link

@louisinger louisinger commented Oct 27, 2021

The CoinosProvider will be injected into web pages by Marina extension. The idea is to provide methods in order to interact with the 2of2 (Marina+coinos) account.

  • getNetwork, getNextAddress and getNextChangeAddress, signTransaction, sendTransaction behave like the marina provider. Note that the addresses in getNext*Address methods will use the multisig scripts (2of2 account).
  • getCoins, getTransactions and getBalances returns data about coins owned by the 2of2 account, excluding all other coins owned by the marina main account.
  • approveSpend returns a PSET (base64 encoded) signed by the Marina user. It lets to create PSET with signed inputs using SIGHASH_NONE + SIGHASH_ANYONE_CAN_PAY and no outputs.
  • getWalletInfo returns the xpub of Marina and the xpub used as cosigner (the coinos one).

@asoltys @tiero please review

louisinger and others added 5 commits September 28, 2021 11:43
…emventures#8)

* add public key in SignedMessage type

* add Lint and Build in GH actions

* Apply suggestions from code review

Co-authored-by: Marco Argentieri <[email protected]>
* split index.js into several files + add DataRecipient type

* modify AddressRecipient and DataRecipient
@louisinger louisinger changed the title First coinos interface First version CoinosProvider interface Oct 27, 2021
package.json Outdated Show resolved Hide resolved
@louisinger
Copy link
Author

louisinger commented Oct 29, 2021

Not sure if sendTransaction and signTransaction makes sense here. I think that the user can use the methods provided by marina. The type of "receiver" will define if we'll use Marina singlesig, multisig or both. cc @tiero

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