Skip to content

docs(feedback): apply Tristan's revisions from RequestNetwork/request-api#806#95

Open
rodrigopavezi wants to merge 1 commit into
docs/revamp-kytfrom
docs/revamp-tristan-feedback
Open

docs(feedback): apply Tristan's revisions from RequestNetwork/request-api#806#95
rodrigopavezi wants to merge 1 commit into
docs/revamp-kytfrom
docs/revamp-tristan-feedback

Conversation

@rodrigopavezi
Copy link
Copy Markdown
Member

7 amendments across 3 pages:

  1. Welcome page subtitle rewritten:
    "Request Network protocol allows you to be paid and pay at scale
    without intermediaries across EVM and Tron. Our protocol offers the
    instant payment reconciliation, click-to-pay payment links, and
    cross-chain routing that Web3 payments need."

  2. Welcome page section renamed "Three on-ramps" -> "Three Products"
    with new subtitle pointing at Dashboard, Secure Payment Page, and
    API. Renamed the Secure Payment card to "Secure Payment Page" for
    consistency.

  3. Multi-Chain Checkout: "you fix the destination" -> "you decide of
    the destination" in the Why multi-chain matters section.

  4. Batch Payouts What you'll build: "gas-amortized" -> "gas-sponsored".

  5. Reordered the two batch modes - Hosted Batch Link is now Mode 1
    (Recommended & Safer Approach), Direct Execution is Mode 2
    (Advanced & Self-Secure). Added Tristan's rationale for each.

  6. Cross-chain capability differences spelled out per mode:

    • Hosted Batch Link supports cross-chain execution from a single
      wallet connection with automatic routing across EVM chains.
    • Direct Execution is same-chain only; cross-chain requires one
      transaction per chain on each network's deployed contracts.
  7. Added a wallet-infrastructure compatibility note: Secure Payment
    Page and Dashboard not compatible with Safe multisigs today;
    recommend Utila, DFNS, or Fireblocks for corporate treasury;
    WalletConnect-based infrastructure is fully compatible.

…-api#806

7 amendments across 3 pages:

1. Welcome page subtitle rewritten:
   "Request Network protocol allows you to be paid and pay at scale
   without intermediaries across EVM and Tron. Our protocol offers the
   instant payment reconciliation, click-to-pay payment links, and
   cross-chain routing that Web3 payments need."

2. Welcome page section renamed "Three on-ramps" -> "Three Products"
   with new subtitle pointing at Dashboard, Secure Payment Page, and
   API. Renamed the Secure Payment card to "Secure Payment Page" for
   consistency.

3. Multi-Chain Checkout: "you fix the destination" -> "you decide of
   the destination" in the Why multi-chain matters section.

4. Batch Payouts What you'll build: "gas-amortized" -> "gas-sponsored".

5. Reordered the two batch modes - Hosted Batch Link is now Mode 1
   (Recommended & Safer Approach), Direct Execution is Mode 2
   (Advanced & Self-Secure). Added Tristan's rationale for each.

6. Cross-chain capability differences spelled out per mode:
   - Hosted Batch Link supports cross-chain execution from a single
     wallet connection with automatic routing across EVM chains.
   - Direct Execution is same-chain only; cross-chain requires one
     transaction per chain on each network's deployed contracts.

7. Added a wallet-infrastructure compatibility note: Secure Payment
   Page and Dashboard not compatible with Safe multisigs today;
   recommend Utila, DFNS, or Fireblocks for corporate treasury;
   WalletConnect-based infrastructure is fully compatible.
This was referenced May 11, 2026
@rodrigopavezi rodrigopavezi marked this pull request as ready for review May 11, 2026 13:59
Copy link
Copy Markdown
Member Author

rodrigopavezi commented May 11, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 11, 2026

Greptile Summary

This PR applies editorial revisions across three docs pages: a rewritten welcome subtitle, a renamed "Three Products" section, reordered batch-payout modes (Hosted Batch Link promoted to Mode 1/recommended), per-mode cross-chain capability notes, and a new wallet-infrastructure compatibility callout for Safe multisig users.

  • use-cases/multi-chain-checkout.mdx: "you fix the destination" is replaced with "you decide of the destination" — the new phrasing contains a grammatical error ("decide of" is not idiomatic English).
  • use-cases/batch-payouts.mdx: Mode order and guidance updated correctly, but all three real-world code examples still use the Direct Execution path (now labeled "Advanced"), creating a mixed signal for developers reading the examples first; a minor word-repetition ("hosted…hosted") also slipped in.
  • use-cases/welcome.mdx: Section rename and subtitle refresh look good; the new subtitle incorrectly implies the Dashboard and Secure Payment Page are accessed "through the Request Network SDK".

Confidence Score: 3/5

The structural changes are sound, but a grammatical error was introduced into the multi-chain page and the real-world examples contradict the newly recommended mode.

The "you decide of the destination" phrasing in multi-chain-checkout.mdx leaves a grammatically broken sentence in public-facing documentation. Additionally, all three real-world code examples in batch-payouts.mdx demonstrate the path now labeled "Advanced & Self-Secure", sending an implicit message opposite to the page's updated recommendation.

use-cases/multi-chain-checkout.mdx (grammar error) and use-cases/batch-payouts.mdx (example code contradicts newly-promoted recommended path)

Important Files Changed

Filename Overview
use-cases/batch-payouts.mdx Modes reordered (Hosted Batch Link is now Mode 1/recommended, Direct Execution is Mode 2/advanced); cross-chain capabilities documented per mode; wallet infrastructure compatibility note added; but "hosted…hosted" redundancy, and all real-world examples still demonstrate the non-recommended Direct Execution path.
use-cases/multi-chain-checkout.mdx Single line change replacing "you fix" with "you decide of" — introduces a grammatical error ("decide of" is not standard English; should be "decide on").
use-cases/welcome.mdx Welcome page subtitle rewritten, section renamed "Three Products", card renamed "Secure Payment Page"; the new subtitle inaccurately says users interact with "Request Network SDK" through all three products when Dashboard and Secure Payment Page are no-code tools unrelated to the SDK.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Need to pay multiple recipients?] --> B{Who controls the payer wallet?}
    B -->|Human / Finance team| C[Mode 1: Hosted Batch Link\nRecommended & Safer]
    B -->|Automated backend| D{Need cross-chain payouts?}
    D -->|Yes| C
    D -->|No, same-chain only| E[Mode 2: Direct Execution\nAdvanced & Self-Secure]
    C --> F[POST /v2/secure-payments]
    F --> G[Returns pay.request.network URL]
    G --> H[Payer reviews and signs once]
    E --> J[POST /v2/payouts/batch]
    J --> K[Returns calldata]
    K --> L[Backend broadcasts transactions]
Loading

Comments Outside Diff (1)

  1. use-cases/batch-payouts.mdx, line 134-191 (link)

    P2 Real-world examples all use the non-recommended mode

    All three "Real-world examples" (Marketplace seller payouts, Refund campaign, Contractor payroll) call POST /v2/payouts/batch — the Direct Execution path now labeled "Advanced & Self-Secure". This contradicts the recommendation added earlier in the page that Hosted Batch Link should be the default choice for most scenarios. A reader who skims to the examples section will get the opposite signal from what the restructured modes section intends to convey.

Reviews (1): Last reviewed commit: "docs(feedback): apply Tristan's revision..." | Re-trigger Greptile

Comment thread use-cases/multi-chain-checkout.mdx
Comment thread use-cases/batch-payouts.mdx
Comment thread use-cases/welcome.mdx
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.

1 participant