Skip to content

feat(abstract-utxo): add zec address codec with unified address support - #9725

Open
veetragjain wants to merge 1 commit into
masterfrom
veetragjain/cshld-1724-add-address-codec-with-zip-316-unified-address-support
Open

feat(abstract-utxo): add zec address codec with unified address support#9725
veetragjain wants to merge 1 commit into
masterfrom
veetragjain/cshld-1724-add-address-codec-with-zip-316-unified-address-support

Conversation

@veetragjain

Copy link
Copy Markdown
Contributor

Ticket: CSHLD-1639

@linear-code

linear-code Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

CSHLD-1724

@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

⚠️ Unit tests are failing on Node 26.x (Current release line, non-blocking). This is not an LTS version yet, so it does not block merge, but it signals an incompatibility to fix before Node 26.x becomes LTS.

View run

@veetragjain
veetragjain marked this pull request as ready for review September 11, 2026 14:32
@veetragjain
veetragjain requested a review from a team as a code owner September 11, 2026 14:32
* (plain transparent, transparent-only UA, malformed, wrong network)
* throws, since it has no Orchard receiver to resolve.
*/
override decode(address: string, unifiedRecipientPreference: UnifiedRecipientPreference = 'transparent'): Uint8Array {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't this be a constructor arg / class member instead? then you can pass in AddressCodec instances and the callers don't need to know the preference

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed it to constructor arg

@veetragjain
veetragjain force-pushed the veetragjain/cshld-1724-add-address-codec-with-zip-316-unified-address-support branch from 75f35e7 to 6bfe3a7 Compare September 11, 2026 16:17
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