Page MenuHomePhabricator

[Cashtab] Use ecash-script lib for op return tx parsing
ClosedPublic

Authored by bytesofman on Jan 26 2024, 13:51.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC70a5b48b4769: [Cashtab] Use ecash-script lib for op return tx parsing
Summary

Improve handling of OP_RETURN parsing in Cashtab tx history by using the available getStackArray method from ecash-script.

Delete the now-unused parseOpReturn method and its associated tests and mocks.

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien requested changes to this revision.Jan 26 2024, 15:35
Fabien added a subscriber: Fabien.
Fabien added inline comments.
cashtab/package.json
3 ↗(On Diff #44620)

?

This revision now requires changes to proceed.Jan 26 2024, 15:35
cashtab/package.json
3 ↗(On Diff #44620)

this will always need to be bumped after rebasing to the latest master.

i'm thinking about a way to automate this but not sure of the best approach.

mb run a script that always minor-bumps it on landing if it has not otherwise been changed? Is this the kind of thing that can be done in CI?

version bump but might need to happen again before landing

Fabien added inline comments.
cashtab/package.json
3 ↗(On Diff #44620)

it was bumping 2 versions

This revision is now accepted and ready to land.Jan 26 2024, 17:15