Skip to content

Conversation

max-koro
Copy link
Contributor

No description provided.

Copy link

linear bot commented Aug 11, 2025

Copy link

netlify bot commented Aug 11, 2025

Deploy Preview for incomparable-tiramisu-91a96a ready!

Name Link
🔨 Latest commit b1332d0
🔍 Latest deploy log https://app.netlify.com/projects/incomparable-tiramisu-91a96a/deploys/689c983b83def4000852b8bb
😎 Deploy Preview https://deploy-preview-381--incomparable-tiramisu-91a96a.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.


Clearing usually occurs one to three days after the authorization, depending on the merchant.
The process is not absolute, however, so make sure to review the examples section for several situational flows.
The process is not absolute, however, so make sure to review the [examples](#examples) section for several situational flows.
Copy link
Contributor

Choose a reason for hiding this comment

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

The 'however' in this sentence feels misplaced and redundant. Can we just say its not absolute, and to make sure they review?

@@ -284,6 +284,8 @@ Consider a cardholder who orders and returns a package, all online:
When items are returned in-store, they can be hard to link back to the debit.
When a merchant needs to refund a customer but can't connect the refund transaction to the original debit, they create a new credit transaction.

When merchants correctly reference the original debit authorization, Swan will link the `CardOutCredit` refund transaction to the same `paymentId` as the original authorization.
Copy link
Contributor

Choose a reason for hiding this comment

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

Something is missing here - Cardoutcredit isn't the refund transaction is it? It's a transaction type that is used when processing refunds. This needs a little rewording.

Copy link
Contributor

@racene-swan racene-swan left a comment

Choose a reason for hiding this comment

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

Good spotting on all the typos! The section itself needs a reword for clarity.

Copy link
Contributor

@racene-swan racene-swan left a comment

Choose a reason for hiding this comment

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

Parfait!

@max-koro max-koro merged commit f82cc91 into main Aug 18, 2025
4 checks passed
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