Skip to content

feat: unified purchaseToken and fix transaction ID issues#523

Merged
hyochan merged 6 commits intomainfrom
feat/unified-purchase-token-and-transaction-fixes
Aug 18, 2025
Merged

feat: unified purchaseToken and fix transaction ID issues#523
hyochan merged 6 commits intomainfrom
feat/unified-purchase-token-and-transaction-fixes

Conversation

@hyochan
Copy link
Copy Markdown
Owner

@hyochan hyochan commented Aug 17, 2025

Changes

  • Unify purchaseToken field for both iOS (JWS) and Android (purchase token)
  • Fix transaction ID issues by removing StoreKit Configuration
  • Prevent duplicate transaction events
  • Add comprehensive error handling
  • Deprecate platform-specific token fields with migration path

Benefits

  • Cross-platform server validation with single purchaseToken field
  • Proper secure transaction IDs in Sandbox/Production
  • Better error handling and duplicate event prevention

- Add unified purchaseToken field for both iOS (JWS) and Android (purchase token)
- Deprecate jwsRepresentationIOS and purchaseTokenAndroid in favor of purchaseToken
- Fix duplicate transaction events by adding processed transaction tracking
- Remove StoreKit Configuration to use proper Sandbox transaction IDs
- Fix iOS transaction ID issue showing sequential numbers instead of secure IDs
- Add comprehensive error handling following expo-iap patterns
- Update documentation for unified purchaseToken and transaction troubleshooting
- Remove orphaned darwin directory and fix pubspec configuration
…okens

- Update types.md with unified purchaseToken field documentation
- Add deprecation notices for jwsRepresentationIOS and purchaseTokenAndroid
- Document transaction ID formats across different environments
- Add troubleshooting guide for transaction ID and purchase token issues
- Include migration examples for v6.0.0 unified token usage
@hyochan hyochan added the 🎯 feature New feature label Aug 17, 2025
@hyochan hyochan force-pushed the feat/unified-purchase-token-and-transaction-fixes branch from 623750c to cefbf1d Compare August 17, 2025 21:24
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 17, 2025

Codecov Report

❌ Patch coverage is 33.50785% with 127 lines in your changes missing coverage. Please review.
✅ Project coverage is 21.37%. Comparing base (e92746e) to head (3b0baea).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
lib/types.dart 28.07% 123 Missing ⚠️
lib/flutter_inapp_purchase.dart 80.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #523      +/-   ##
==========================================
+ Coverage   15.18%   21.37%   +6.19%     
==========================================
  Files           6        6              
  Lines        1337     1511     +174     
==========================================
+ Hits          203      323     +120     
- Misses       1134     1188      +54     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

- Add strict formatting validation with --set-exit-if-changed flag
- Include step-by-step pre-commit process matching CI behavior
- Fix test file formatting issues
@hyochan hyochan force-pushed the feat/unified-purchase-token-and-transaction-fixes branch from fdd6315 to 1c9f01b Compare August 17, 2025 23:42
- Add tests for Android and iOS purchaseToken handling
- Test purchaseToken field for both platforms in PurchasedItem
- Improve test coverage for new unified token functionality
- Address Codecov coverage requirements for recent changes
@hyochan hyochan force-pushed the feat/unified-purchase-token-and-transaction-fixes branch from 1c9f01b to 3b0baea Compare August 18, 2025 00:10
@hyochan hyochan merged commit 9856f1f into main Aug 18, 2025
3 checks passed
@hyochan hyochan deleted the feat/unified-purchase-token-and-transaction-fixes branch August 18, 2025 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎯 feature New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant