Page MenuHomePhabricator

[ecash-agora] Partial Agora offers: Add ALP support to plugin, `Agora` and `AgoraOffer`
ClosedPublic

Authored by tobias_ruck on Sep 12 2024, 23:27.

Details

Summary

In D16745 we added support for SLP, in this diff we add support for ALP.

The biggest difference is that offers use the OP_RETURN to advertize instead of the scriptSig like in SLP.

Depends on D16754.

Test Plan

npm run integration-tests

Diff Detail

Repository
rABC Bitcoin ABC
Branch
ecash-agora-partial-offer-alp
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 30275
Build 60077: Build Diffecash-agora-tests · ecash-agora-integration-tests
Build 60076: arc lint + arc unit

Event Timeline

tobias_ruck retitled this revision from [ecash-agora] Partial Agora offers: Add SLP support to plugin, `Agora` and `AgoraOffer` to [ecash-agora] Partial Agora offers: Add ALP support to plugin, `Agora` and `AgoraOffer`.Sep 13 2024, 10:22
bytesofman added inline comments.
modules/ecash-agora/src/agora.ts
259 ↗(On Diff #49660)
modules/ecash-agora/tests/partial.alp.bigsats.test.ts
334 ↗(On Diff #49660)

applies to this stack in general -- why do we not test the 0-index output? it does need to be exact for the tx to be valid, right?

This revision now requires changes to proceed.Sep 18 2024, 21:34
This revision is now accepted and ready to land.Sep 24 2024, 19:56