Skip to content

Paywalls Tester: group live paywalls by template#3276

Merged
NachoSoto merged 2 commits into
mainfrom
paywalls-tester-group-by-template
Oct 4, 2023
Merged

Paywalls Tester: group live paywalls by template#3276
NachoSoto merged 2 commits into
mainfrom
paywalls-tester-group-by-template

Conversation

@NachoSoto

@NachoSoto NachoSoto commented Oct 4, 2023

Copy link
Copy Markdown
Contributor

image

@NachoSoto NachoSoto added the test label Oct 4, 2023
@NachoSoto NachoSoto requested a review from a team October 4, 2023 19:59
@NachoSoto

Copy link
Copy Markdown
Contributor Author

I recommend reviewing this ignoring whitespaces.

let offerings = try await Purchases.shared.offerings()
.all
.map(\.value)
.sorted { $0.serverDescription > $1.serverDescription }

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we can delete this sorting now, right? we have it again on line 51

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.

That one is sorting keys (templates). This is sorting the values (offerings) by name.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ohh, sorry, makes sense.

@NachoSoto NachoSoto enabled auto-merge (squash) October 4, 2023 20:23
@NachoSoto NachoSoto merged commit f376430 into main Oct 4, 2023
@NachoSoto NachoSoto deleted the paywalls-tester-group-by-template branch October 4, 2023 20:38
NachoSoto pushed a commit that referenced this pull request Oct 6, 2023
**This is an automatic release.**

### RevenueCatUI
* `Paywalls`: improved purchase-in-progress UI (#3279) via NachoSoto
(@NachoSoto)
### Bugfixes
* `SK2StoreProduct.priceFormatter`: use locale from `StoreKit.Product`
(#3278) via NachoSoto (@NachoSoto)
### Performance Improvements
* `AAAttribution.attributionToken`: avoid using on main thread (#3281)
via NachoSoto (@NachoSoto)
### Other Changes
* `Paywalls Tester`: group live paywalls by template (#3276) via
NachoSoto (@NachoSoto)
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