Skip to content

Use SK1 Receipt for isSandbox #6247

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
fire-at-will wants to merge 46 commits into
base: main
Choose a base branch
from
Draft

Use SK1 Receipt for isSandbox #6247

fire-at-will wants to merge 46 commits into from
+782 −126

Conversation

@fire-at-will
Copy link
Contributor

@fire-at-will fire-at-will commented Feb 11, 2026

Checklist

  • If applicable, unit tests
  • If applicable, create follow-up issues for purchases-android and hybrids

Motivation

Description

ajpallares and others added 30 commits January 29, 2026 14:49
@ajpallares
Use AppTransaction to get environment
920d3f0
@ajpallares
Renames
60bb269
@ajpallares
Merge branch 'main' into use-app-transaction-for-environment
8795ce8
@ajpallares
Merge branch 'main' into use-app-transaction-for-environment
226a4a4
@ajpallares
Simplification
64e2a90
@ajpallares
More tests
3fd3f9b
@ajpallares
Add tests on mac
1f3b71c
@ajpallares
Add more tests for behavior before prefetch finishes
8acebda
@ajpallares
Fix public Purchases.shared.isSandbox to not cache the information …
800a92b 
…(now it's only cached by `SandboxEnvironmentDetector`)
@ajpallares
Merge branch 'main' into use-app-transaction-for-environment
ae6ad14
@ajpallares
Merge branch 'main' into use-app-transaction-for-environment
3b89a94
@ajpallares
Try this
a7c0778
@ajpallares
TEMP: test this (temporary disable it)
007f227
@ajpallares
temp: run-installation-tests CI parameter
d859574
@ajpallares
Remove duplication
ca24312
@ajpallares
TEMP: disable task
349656c
@ajpallares
Bring back Atomic for Bundle
16fb9a3
@ajpallares
TEMP: Test directly SK api + no Atomic
7d01ebc
@ajpallares
TEMP: Test this (no SK)
0cce26b
@ajpallares
TEMP fix
e358952
@ajpallares
try with long wait time
4034cef
@ajpallares
Force try to make sure
641d686
@ajpallares
Now with AppTransaction.shared
1c73095
@ajpallares
Atomic cachedAppTransactionEnvironment
159d81b
@ajpallares
Try background
3816411
@ajpallares
Try long timeout
c3dfa67
@ajpallares
Fix
5f233cf
@ajpallares
Revert "Try long timeout"
774b318 
This reverts commit c3dfa67.
@ajpallares
TEMP: only spm installation tests
3e87541
@ajpallares
actual implementation
18dc976
ajpallares and others added 13 commits January 30, 2026 12:01
@ajpallares
test no background in Task.detached
55f1c5f
@ajpallares
Check this
827d210
@ajpallares
Check this now
33c1f60
@ajpallares
This seems to be working
5b3a639
@ajpallares
Reduce flakiness
d98e118
@fire-at-will
Fix AppTransaction Environment Detection Test Flakiness on MacOS (#6173)
1ceb404
@fire-at-will
Merge branch 'main' into use-app-transaction-for-environment
dc6ee86
@ajpallares
Make appTransactionEnvironment iOS 16+
4d62a16
@fire-at-will
Fix iOS 14 + 15 Tests (#6174)
3f8a896 
* [TEMP]: only run the CI that we want to run

* only await when AppTransaction is available

* decrease timeout from 10s -> 3s

* dont run tests when on ios version that they shouldnt be run on

* swiftlint

* test fixes

* compilation fix

* test fix

* Revert "[TEMP]: only run the CI that we want to run"

This reverts commit 9aa0b2c.
@fire-at-will
Make Installation tests async await (#6178)
05f8b8d 
* make carthage installation tests async

* [TEMP]: run carthage tests
@ajpallares
Inject SandboxEnvironmentDetector for integration test (#6177)
e553474
@fire-at-will
use SK1 receipt environment value for isSandbox
85de6e1
@fire-at-will
test fixes
da0e545
@fire-at-will fire-at-will added the pr:fix A bug fix label Feb 11, 2026
@emerge-tools
Copy link

emerge-tools bot commented Feb 11, 2026 •
edited
Loading

📸 Snapshot Test

Error generating snapshot comparison

Name Error message
RevenueCat
com.revenuecat.PaywallsTester		 Timeout extracting previews

🛸 Powered by Emerge Tools

@fire-at-will
Copy link
Contributor Author

fire-at-will commented Feb 11, 2026

@RCGitBot please test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

pr:fix A bug fix

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fire-at-will @ajpallares