Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

Test transactions while sending#1182

Draft
yashrajd wants to merge 4 commits intoBitcoinDesign:master from
yashrajd:test-transactions-while-sending
Draft

Test transactions while sending #1182
yashrajd wants to merge 4 commits intoBitcoinDesign:master from
yashrajd:test-transactions-while-sending

Conversation

@yashrajd
Copy link
Contributor

@yashrajd yashrajd commented Jun 26, 2025 •
edited
Loading

Copy link

netlify bot commented Jun 26, 2025 •
edited
Loading

✅ Deploy Preview for bitcoin-design-site ready!

Name Link
🔨 Latest commit 863eab0
🔍 Latest deploy log https://app.netlify.com/projects/bitcoin-design-site/deploys/685e7243b4c12b0008085c7a
😎 Deploy Preview https://deploy-preview-1182--bitcoin-design-site.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@GBKS GBKS requested a review from Copilot June 27, 2025 05:58
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the sending guide to introduce a new "Test transactions" section that explains how users can send small test payments to verify large or unfamiliar transactions.

  • Added a new images block ("imagesTestTransactions") including four image entries.
  • Included a detailed "Test transactions" section outlining the interaction flow for test transactions.
Comments suppressed due to low confidence (1)

guide/daily-spending-wallet/sending.md:271

  • [nitpick] Consider revising this statement for clarity. For example, change it to 'application presents this option to the user on the review screen' and adjust the conditional clause for improved readability.
1. **application presents this option user on review screen**: in the appropriate scenario, such as large amount or new silent payments address is being used.

Comment on lines +141 to +153
alt: Wallet home screen with amount input, pay and request options
caption: Home screens provide quick access to entering payment flows.
- file: send-test-transaction-confirmation
modalImage: send-test-transaction-confirmation-big
alt: Wallet home screen with amount input, pay and request options
caption: Home screens provide quick access to entering payment flows.
- file: home-test-transaction-reminder
modalImage: home-test-transaction-reminder-big
alt: Wallet home screen with amount input, pay and request options
caption: Home screens provide quick access to entering payment flows.
- file: finalise-test-transaction
modalImage: finalise-test-transaction-big
alt: Wallet home screen with amount input, pay and request options
Copy link

Copilot AI Jun 27, 2025

Choose a reason for hiding this comment

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

[nitpick] The alt text for multiple images in the test transactions block is identical, which could hinder accessibility. Consider providing unique, descriptive alt texts for each image to better convey their distinct content.

Suggested change
alt: Wallet home screen with amount input, pay and request options
caption: Home screens provide quick access to entering payment flows.
- file: send-test-transaction-confirmation
modalImage: send-test-transaction-confirmation-big
alt: Wallet home screen with amount input, pay and request options
caption: Home screens provide quick access to entering payment flows.
- file: home-test-transaction-reminder
modalImage: home-test-transaction-reminder-big
alt: Wallet home screen with amount input, pay and request options
caption: Home screens provide quick access to entering payment flows.
- file: finalise-test-transaction
modalImage: finalise-test-transaction-big
alt: Wallet home screen with amount input, pay and request options
alt: Review screen showing a toggle to enable test transactions
caption: Home screens provide quick access to entering payment flows.
- file: send-test-transaction-confirmation
modalImage: send-test-transaction-confirmation-big
alt: Confirmation screen for sending a test transaction
caption: Home screens provide quick access to entering payment flows.
- file: home-test-transaction-reminder
modalImage: home-test-transaction-reminder-big
alt: Reminder on the home screen about an ongoing test transaction
caption: Home screens provide quick access to entering payment flows.
- file: finalise-test-transaction
modalImage: finalise-test-transaction-big
alt: Finalization screen for completing a test transaction

Copilot uses AI. Check for mistakes.

{% include image-gallery.html pages = page.imagesReview %}

### Test transactions
Copy link
Collaborator

@swedishfrenchpress swedishfrenchpress Jun 27, 2025

Choose a reason for hiding this comment

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

I notice this section isn't showing up in the table of contents on the preview. Could we fix that?

Technically, this feature can utilise [replace-by-fee](https://bitcoinops.org/en/topics/replace-by-fee/) (RBF) or a simple small-value transaction.

There are three main interactions in this feature:
1. **application presents this option user on review screen**: in the appropriate scenario, such as large amount or new silent payments address is being used.
Copy link
Collaborator

@swedishfrenchpress swedishfrenchpress Jun 27, 2025

Choose a reason for hiding this comment

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

Capitalize the A in application.


There are three main interactions in this feature:
1. **application presents this option user on review screen**: in the appropriate scenario, such as large amount or new silent payments address is being used.
2. **user chooses & sends the test transaction**: they may ignore it if they have no way of communicating with the receiver. Inform users about relevant fees, if any.
Copy link
Collaborator

@swedishfrenchpress swedishfrenchpress Jun 27, 2025

Choose a reason for hiding this comment

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

  1. User chooses & sends the test transaction: they may ignore it if they have no way of communicating with the receiver. Inform users about relevant fees, if any.

There are three main interactions in this feature:
1. **application presents this option user on review screen**: in the appropriate scenario, such as large amount or new silent payments address is being used.
2. **user chooses & sends the test transaction**: they may ignore it if they have no way of communicating with the receiver. Inform users about relevant fees, if any.
3. **user finalises the original payment**: allow the user to easily find and finalise the transaction. Applications can even remind the user about this after appropriate time.
Copy link
Collaborator

@swedishfrenchpress swedishfrenchpress Jun 27, 2025

Choose a reason for hiding this comment

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

  1. User finalises the original payment: allow the user to easily find and finalise the transaction. Applications can even remind the user about this after appropriate time.

Technically, this feature can utilise [replace-by-fee](https://bitcoinops.org/en/topics/replace-by-fee/) (RBF) or a simple small-value transaction.

There are three main interactions in this feature:
1. **application presents this option user on review screen**: in the appropriate scenario, such as large amount or new silent payments address is being used.
Copy link
Collaborator

@swedishfrenchpress swedishfrenchpress Jun 27, 2025

Choose a reason for hiding this comment

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

  1. Application presents this option user on review screen: in the appropriate scenario, such as large amount or new silent payments address is being used.

Copy link
Collaborator

I left a few minor comments. On the whole I'm not sure how I feel about this addition. I think test transactions are very useful and I usually do them when I'm getting familiar with a new wallet, but I'm conflicted about encouraging wallets to add this into their flow. If we follow the KISS (Keep It Simple Stupid) principle it seems like this adds extra complexity. I know it's an optional feature, and that's emphasized in the copy, so maybe that's enough. Just my 2 sats on this.
oli28 reacted with thumbs up emoji

Copy link
Collaborator

Really good Lopp thread on test transaction dropped this weekend. Some food for thought on this practice.

https://x.com/lopp/status/1941489877046211054

yashrajd reacted with thumbs up emoji

Copy link
Contributor

GBKS commented Sep 30, 2025

Aloha. Last updated here was July 5. Anything you need to keep moving forward?
yashrajd reacted with eyes emoji

Copy link
Contributor Author

yashrajd commented Oct 2, 2025

Aloha. Last updated here was July 5. Anything you need to keep moving forward?

I'll review this and get back, but Oct is super-busy so I might not be able to get to it right now. More community feedback is always welcome.
GBKS reacted with thumbs up emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@swedishfrenchpress swedishfrenchpress swedishfrenchpress left review comments

Copilot code review Copilot Copilot left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Comments

AltStyle によって変換されたページ (->オリジナル) /