Skip to content

fix(receipt): preserve subscriptionId metadata - #36

Open
EfeDurmaz16 wants to merge 1 commit into
stripe:mainfrom
EfeDurmaz16:fix/receipt-subscription-id
Open

fix(receipt): preserve subscriptionId metadata#36
EfeDurmaz16 wants to merge 1 commit into
stripe:mainfrom
EfeDurmaz16:fix/receipt-subscription-id

Conversation

@EfeDurmaz16

Copy link
Copy Markdown
Contributor

What

  • Add optional subscription_id support to Mpp::Receipt.
  • Preserve subscriptionId when parsing and serializing Payment-Receipt.
  • Add regression coverage for foreign receipts and round trips.

Why

Subscription activation and renewal receipts can carry a server-issued subscriptionId. Ruby currently drops this field during parsing, while the Python, Go, and Rust SDKs preserve it.

How

Map Ruby's optional subscription_id field to the wire-level subscriptionId field. Existing receipts remain compatible, and this PR does not implement the subscription intent itself.

Verification

  • bundle exec rake test: 336 tests, 836 assertions
  • bundle exec standardrb
  • bundle exec srb tc

Not included

  • Subscription activation or renewal
  • Access-key authorization
  • Subscription stores or framework APIs

@EfeDurmaz16
EfeDurmaz16 marked this pull request as ready for review July 20, 2026 14:39
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