Skip to content

Galaxy Scaffolding Classes#2901

Merged
fire-at-will merged 6 commits into
samsung-devfrom
samsung-skeleton-structure
Dec 8, 2025
Merged

Galaxy Scaffolding Classes#2901
fire-at-will merged 6 commits into
samsung-devfrom
samsung-skeleton-structure

Conversation

@fire-at-will

@fire-at-will fire-at-will commented Dec 5, 2025

Copy link
Copy Markdown
Contributor

Description

Introduces the scaffolding for several classes that will be foundational to Samsung development as we move forward:

  • GalaxyBillingWrapper: implementation of BillingAbstract. The class currently satisfies all of the BillingAbstract's requirements, but all functions are stubbed with TODOs
  • GalaxyPurchasingData: will be used when processing samsung purchases
  • GalaxyStoreProduct: represents a product from the Samsung store

@fire-at-will fire-at-will requested a review from a team as a code owner December 5, 2025 21:21
@fire-at-will fire-at-will added the pr:feat A new feature label Dec 5, 2025
@codecov

codecov Bot commented Dec 5, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 52.17391% with 22 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.36%. Comparing base (8151db5) to head (91a2d27).
⚠️ Report is 4 commits behind head on samsung-dev.

Files with missing lines Patch % Lines
...evenuecat/purchases/galaxy/GalaxyBillingWrapper.kt 0.00% 13 Missing ⚠️
...evenuecat/purchases/galaxy/GalaxyPurchasingData.kt 0.00% 5 Missing ⚠️
.../revenuecat/purchases/galaxy/GalaxyStoreProduct.kt 85.71% 3 Missing and 1 partial ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##           samsung-dev    #2901      +/-   ##
===============================================
- Coverage        78.46%   78.36%   -0.10%     
===============================================
  Files              332      335       +3     
  Lines            12880    12926      +46     
  Branches          1749     1751       +2     
===============================================
+ Hits             10106    10130      +24     
- Misses            2040     2061      +21     
- Partials           734      735       +1     

☔ 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.

@fire-at-will fire-at-will changed the base branch from main to samsung-dev December 5, 2025 22:04
override val name: String,

/**
* Name of the product. This will match the name for Galaxy products.

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.

This is not a typo: ProductVos do not have a title property, so we'll need to just put the name property in here (or leave it as a blank string)

@fire-at-will fire-at-will changed the title Samsung Scaffolding Classes Galaxy Scaffolding Classes Dec 8, 2025
@fire-at-will fire-at-will merged commit 42f1a7e into samsung-dev Dec 8, 2025
23 checks passed
@fire-at-will fire-at-will deleted the samsung-skeleton-structure branch December 8, 2025 14:38
import com.revenuecat.purchases.models.PurchasingData
import dev.drewhamilton.poko.Poko

sealed class GalaxyPurchasingData : PurchasingData {

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.

Same as in the preview PR, might make sense to make these experimental... but NABD since we're merging to an integration branch

override val price: Price,

/**
* Name of the product. This will match the name for Galaxy products.

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.

A bit weird doc, saying that it will match itself :P Probably not needed? Like

Suggested change
* Name of the product. This will match the name for Galaxy products.
* Name of the product.

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

Labels

pr:feat A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants