Skip to content

Remove unnecessary appInBackground parameters#1508

Merged
vegaro merged 7 commits into
mainfrom
remove-appinbackground-everywhere
Nov 30, 2023
Merged

Remove unnecessary appInBackground parameters#1508
vegaro merged 7 commits into
mainfrom
remove-appinbackground-everywhere

Conversation

@vegaro

@vegaro vegaro commented Nov 30, 2023

Copy link
Copy Markdown
Member

With #1502 we don't need to pass appInBackground as a parameter to the functions in BillingAbstract.

This PR removes those parameters.

@vegaro vegaro requested a review from a team November 30, 2023 16:02

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

Nice!

@vegaro vegaro enabled auto-merge (squash) November 30, 2023 16:19
@vegaro vegaro merged commit 0aa0651 into main Nov 30, 2023
@vegaro vegaro deleted the remove-appinbackground-everywhere branch November 30, 2023 16:21
This was referenced Dec 5, 2023
vegaro pushed a commit that referenced this pull request Dec 5, 2023
**This is an automatic release.**

### RevenueCatUI
* Paywalls: Add `PaywallFooterView` (#1509) via Toni Rico (@tonidero)
* Paywalls: Remove `PaywallActivity` theme to pickup application's theme
by default (#1511) via Toni Rico (@tonidero)
* Paywalls: Auto-close paywall activity if restore grants required
entitlement identifier (#1507) via Toni Rico (@tonidero)
### Bugfixes
* Improve pricePerYear price calculation precision (#1515) via Toni Rico
(@tonidero)
* Improve price per month accuracy for weekly subscriptions (#1504) via
Andy Boedo (@aboedo)
### Dependency Updates
* Bump danger from 9.4.0 to 9.4.1 (#1512) via dependabot[bot]
(@dependabot[bot])
### Other Changes
* Remove unnecessary appInBackground parameters (#1508) via Cesar de la
Vega (@vegaro)
* Create `PurchasesStateProvider` (#1502) via Cesar de la Vega (@vegaro)

Co-authored-by: revenuecat-ops <ops@revenuecat.com>
@codecov

codecov Bot commented Sep 17, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.36%. Comparing base (ba4d880) to head (0f46583).
Report is 250 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1508      +/-   ##
==========================================
- Coverage   84.50%   84.36%   -0.15%     
==========================================
  Files         218      218              
  Lines        7211     7177      -34     
  Branches     1004     1004              
==========================================
- Hits         6094     6055      -39     
- Misses        729      734       +5     
  Partials      388      388              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants