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 message support #504

Closed
6 tasks
pixelcircuits opened this issue Sep 10, 2022 · 1 comment · Fixed by #515
Closed
6 tasks

Add message support #504

pixelcircuits opened this issue Sep 10, 2022 · 1 comment · Fixed by #515
Assignees
Labels
feat Issue is a feature
Milestone

Comments

@pixelcircuits
Copy link
Contributor

Add support for InputMessage and OutputMessage. As development of the bridge progresses, we will need support for being able to spend and create messages. Refer to the currently implemented spec here, and you can find the parallel rust sdk issue here.

Here is a non exhaustive list of what needs to be done:

  • Add InputMessage to inputs.ts in transactions (then also under provider)
  • OutputWithdrawal needs to be updated to OutputMessage in outputs.ts (then also under provider)
  • Add getMessages to the provider (similar to getCoins)
  • Add addMessage and addMessages to transaction-request.ts (similar to addCoin and addCoins)
    • getCoinInputWitnessIndexByOwner should be modified to support the owner field on input messages
    • can still try to auto add a change output but for the base asset
@camsjams camsjams self-assigned this Sep 10, 2022
@luizstacio luizstacio added this to the v0.17.0 milestone Sep 15, 2022
@camsjams
Copy link
Contributor

Will need to remove owner once fuel-core releases a new version from master, see FuelLabs/fuel-core#600

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat Issue is a feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants