docs(payments): remove stale post-T6 callout, document held payments inline - #874
Open
max-digi wants to merge 1 commit into
Open
docs(payments): remove stale post-T6 callout, document held payments inline#874max-digi wants to merge 1 commit into
max-digi wants to merge 1 commit into
Conversation
…line T6 has been live on testnet and mainnet since June 2026, so the "On post-T6 networks" warning at the top of Send a Payment no longer reflects reality. Remove the callout and fold its guidance into the places where readers act on it: - Send a Payment: receipt step decodes the Transfer log and shows Delivered vs Held; Index payment events covers TransferBlocked; new best-practice section with Failed / Delivered / Held outcomes; learning-resources card for Configure Receive Policies. - Accept a Payment: explain receive policies and ReceivePolicyGuard up front, filter the Transfer watcher by `to`, and add a verification best practice about checking the recipient rather than tx success. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The warning at the top of Send a Payment opens with "On post-T6 networks…". T6 has been active on testnet and mainnet since June 2026 (see T6 Network Upgrade), so the conditional framing is stale. This was the last
post-T6conditional in the guides.The behavior the callout describes is permanent, so rather than delete the guidance this PR removes the callout and moves the content to where readers act on it.
Changes
send-a-payment.mdx:::warningcallout.Transferlog and render "Delivered" vs "Held by receiver policy".TransferBlockedand note that aTransfertoReceivePolicyGuardmeans held, not delivered.accept-a-payment.mdx(receiver-side counterpart, previously silent on receive policies)to, and explainTransfer-to-guard +TransferBlockedas the held signal.Review notes
parseEventLogsfrom viem and the existingAbis.tip20export already imported by the batch recipe. It has not been run through the twoslash build locally; please confirm it type-checks in the docs preview.#understand-receive-policy-behavioranchor targets the existing "Understand receive policy behavior" heading on Configure Receive Policies.🤖 Generated with Claude Code