Skip to content

Conversation

hajjimo
Copy link
Contributor

@hajjimo hajjimo commented Sep 16, 2025

Description of changes

Issue #647 - Guide for making a one-time remittance payment with a fixed send amount (debitAmount)

Required

  • Used LinkOut component on external links
  • Reviewed Vale errors and made changes where appropriate
  • Ran Prettier
  • Previewed updates in Netlify
  • Received SME and/or peer approval if updates are significant
  • Included a "fixes #" comment

Conditional

  • Ensured sequence diagrams follow our style guide
  • Included code samples where appropriate
  • Updated related READMEs

Copy link

netlify bot commented Sep 16, 2025

Deploy Preview for openpayments-preview ready!

Name Link
🔨 Latest commit e8ccb3c
🔍 Latest deploy log https://app.netlify.com/projects/openpayments-preview/deploys/68d4093420a870000853668d
😎 Deploy Preview https://deploy-preview-688--openpayments-preview.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.

```

</details>

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's ask Max and Blair about whether we should add a Step 11 for completing the incoming payment request (optional step)

Copy link
Contributor

@melissahenderson melissahenderson left a comment

Choose a reason for hiding this comment

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

Added comments for a few small things. I think you're ready for a SME review.

@hajjimo hajjimo marked this pull request as ready for review September 24, 2025 11:12
@hajjimo hajjimo requested a review from brad-dow as a code owner September 24, 2025 11:12
hajjimo and others added 2 commits September 24, 2025 14:34
Updated the example response for the recipient's wallet address details
Added link to Create Outgoing Payment API in step 10
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