Skip to content

Comments

feat: implement address on SatsCard#39

Merged
notmandatory merged 1 commit intobitcoindevkit:masterfrom
douglaz:implement_address
Jul 31, 2025
Merged

feat: implement address on SatsCard#39
notmandatory merged 1 commit intobitcoindevkit:masterfrom
douglaz:implement_address

Conversation

@douglaz
Copy link
Contributor

@douglaz douglaz commented Jul 9, 2025

Description

Implement address on SatsCard. Now cktap-cli address works.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

// }
todo!()
pub async fn address(&mut self) -> Result<String, Error> {
// TODO: support testnet
Copy link
Member

Choose a reason for hiding this comment

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

AFAIK SATSCARDs and TAPSIGNERs will never support testnet as part of their security policy to prevent people from tricking users into accepting cards with bogus bitcoin.

@notmandatory notmandatory added the enhancement New feature or request label Jul 31, 2025
Copy link
Member

@notmandatory notmandatory left a comment

Choose a reason for hiding this comment

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

tACK 6ec21f0

Works with my TAPCARDs thanks for the fix.

@notmandatory notmandatory merged commit 3cc6ad8 into bitcoindevkit:master Jul 31, 2025
5 of 6 checks passed
@notmandatory notmandatory mentioned this pull request Aug 7, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants