Skip to content

Fixed snapshots for iOS 12 and 13#1783

Merged
NachoSoto merged 1 commit into
mainfrom
ios12-13-fixes
Jul 29, 2022
Merged

Fixed snapshots for iOS 12 and 13#1783
NachoSoto merged 1 commit into
mainfrom
ios12-13-fixes

Conversation

@NachoSoto

Copy link
Copy Markdown
Contributor

@NachoSoto NachoSoto requested a review from a team July 29, 2022 20:53
@NachoSoto NachoSoto mentioned this pull request Jul 29, 2022

@taquitos taquitos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅🐐
download-68

},
"method" : "POST",
"url" : "https://api.revenuecat.com/v1/receipts"
"url" : "https:\/api.revenuecat.com\/v1\/receipts"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Huh, weird.

@NachoSoto NachoSoto Jul 29, 2022

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's not weird: #1560
We have better formatting for iOS 13+, but that's not available on iOS 12.

@NachoSoto NachoSoto merged commit 0ab0423 into main Jul 29, 2022
@NachoSoto NachoSoto deleted the ios12-13-fixes branch July 29, 2022 21:14
NachoSoto added a commit that referenced this pull request Jul 29, 2022
Follow up to #1783. I couldn't run iOS 12 tests locally (not working with Xcode 14), so I couldn't verify that this was wrong.
Downloading the full `.xcresult` from https://app.circleci.com/pipelines/github/RevenueCat/purchases-ios/7546/workflows/b47843e1-bf9a-40f3-9992-220581772f00/jobs/32130/artifacts (thanks to #1773) helped me see what was wrong.
@NachoSoto NachoSoto mentioned this pull request Jul 29, 2022
NachoSoto added a commit that referenced this pull request Jul 29, 2022
Follow up to #1783. I couldn't run iOS 12 tests locally (not working with Xcode 14), so I couldn't verify that this was wrong.
Downloading the full `.xcresult` from https://app.circleci.com/pipelines/github/RevenueCat/purchases-ios/7546/workflows/b47843e1-bf9a-40f3-9992-220581772f00/jobs/32130/artifacts (thanks to #1773) helped me see what was wrong.
This was referenced Aug 1, 2022
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