Skip to content

docs: document last4/network fields on ApplePayResult#105

Merged
SanthoshCharanBolt merged 3 commits into
mainfrom
SanthoshCharan/apple-pay-readme-last4
Jul 8, 2026
Merged

docs: document last4/network fields on ApplePayResult#105
SanthoshCharanBolt merged 3 commits into
mainfrom
SanthoshCharan/apple-pay-readme-last4

Conversation

@SanthoshCharanBolt

@SanthoshCharanBolt SanthoshCharanBolt commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Description

Follow-up to #101 (already merged), which forwards last4/network from the Apple Pay webview iframe to the merchant's onComplete callback. The README's documented ApplePayResult shape was stale — it never listed network (added in an earlier PR) and didn't get updated for last4 in #101, so the public docs undersold what the SDK actually returns.

Changes:

  • ApplePayResult type reference now lists last4? and network?, with a note that last4 is webview-mode only (native mode's decrypted PassKit payload never carries it).
  • Updated the ApplePay props table's onComplete description to match.

Testing

Docs-only change, no code touched.

Security Review

Important

A security review is required for every PR in this repository to comply with PCI requirements.

  • I have considered and reviewed security implications of this PR and included the summary below.

Security Impact Summary

Docs-only change — no code, data handling, or security-relevant behavior touched.

@snyk-io

snyk-io Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@SanthoshCharanBolt SanthoshCharanBolt self-assigned this Jul 8, 2026
@SanthoshCharanBolt SanthoshCharanBolt marked this pull request as ready for review July 8, 2026 07:28
@SanthoshCharanBolt SanthoshCharanBolt requested a review from a team as a code owner July 8, 2026 07:28
@SanthoshCharanBolt SanthoshCharanBolt enabled auto-merge (squash) July 8, 2026 10:51
@SanthoshCharanBolt SanthoshCharanBolt merged commit 72e76b7 into main Jul 8, 2026
7 checks passed
@SanthoshCharanBolt SanthoshCharanBolt deleted the SanthoshCharan/apple-pay-readme-last4 branch July 8, 2026 17:06
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