Skip to content

[BC5] Cleaning up some TODOs in the code#883

Merged
joshdholtz merged 1 commit into
bc5-supportfrom
bc5-support-picking-off-TODOs
Mar 17, 2023
Merged

[BC5] Cleaning up some TODOs in the code#883
joshdholtz merged 1 commit into
bc5-supportfrom
bc5-support-picking-off-TODOs

Conversation

@joshdholtz

Copy link
Copy Markdown
Member

Motivation

Getting rid of TODOs in the code when this goes into production

Description

  • Added comment to splitting product ids by ":" in purchase tester because probably fine 🤷‍♂️
  • Added javadoc to purchasingData on StoreProduct
  • Removing comment about multi-line purchases because don't need to worry about that (yet)

@joshdholtz joshdholtz requested a review from beylmk March 17, 2023 15:09

@beylmk beylmk 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.

✅ thank you!

@codecov

codecov Bot commented Mar 17, 2023

Copy link
Copy Markdown

Codecov Report

Merging #883 (cb0e713) into bc5-support (3fdf928) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           bc5-support     #883      +/-   ##
===============================================
- Coverage        82.01%   81.99%   -0.02%     
===============================================
  Files              150      150              
  Lines             5349     5349              
  Branches           756      756              
===============================================
- Hits              4387     4386       -1     
  Misses             681      681              
- Partials           281      282       +1     
Impacted Files Coverage Δ
.../main/kotlin/com/revenuecat/purchases/Purchases.kt 85.14% <ø> (ø)

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@joshdholtz joshdholtz merged commit 0c2ff5c into bc5-support Mar 17, 2023
@joshdholtz joshdholtz deleted the bc5-support-picking-off-TODOs branch March 17, 2023 18:43
@joshdholtz joshdholtz mentioned this pull request Mar 23, 2023
joshdholtz added a commit that referenced this pull request Mar 23, 2023
### Breaking Changes
* Rename `introTrial` to `introOffer` on `SubscriptionOptions` (#890)
via Josh Holtz (@joshdholtz)
* Use `rc-ignore-offer` instead of `rc-ignore-befault-offer` (#889) via
Josh Holtz (@joshdholtz)
* AmazonStoreProduct updates (#881) via beylmk (@beylmk)
* Remove Parcelable implementation for classes with ProductDetails as a
member (#866) via beylmk (@beylmk)
* Make Activity first param in PurchaseParams (#882) via Josh Holtz
(@joshdholtz)
### New Features
* Re-enable `observerMode` and `syncPurchases()` (#878) via Josh Holtz
(@joshdholtz)
### Bugfixes
* Re-add `price_string` header on postReceipt (#879) via Josh Holtz
(@joshdholtz)
### Other Changes
* Cleaning up some TODOs in the code (#883) via Josh Holtz (@joshdholtz)
* Amazon Offering Parsing (#824) via beylmk (@beylmk)
* Merge main ->  bc5-support 2023-03-15 (#875) via Toni Rico (@tonidero)
* Fix purchase tester navigation (#871) via beylmk (@beylmk)
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