Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix operator for checking subscription status on first payment attempt. #195

Merged

Conversation

rvdsteege
Copy link
Member

Fix #194.

@rvdsteege rvdsteege linked an issue Sep 19, 2024 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Sep 19, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 22.43%. Comparing base (6bb8284) to head (3c8d63e).
Report is 67 commits behind head on main.

Files with missing lines Patch % Lines
src/Subscriptions/SubscriptionsModule.php 0.00% 1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #195      +/-   ##
============================================
+ Coverage     21.89%   22.43%   +0.53%     
+ Complexity     2530     2520      -10     
============================================
  Files           107      105       -2     
  Lines         10380    10129     -251     
============================================
- Hits           2273     2272       -1     
+ Misses         8107     7857     -250     

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

@rvdsteege rvdsteege merged commit 8f850ce into main Sep 20, 2024
19 of 20 checks passed
@rvdsteege rvdsteege deleted the 194-active-subscription-put-on-hold-when-first-payment-expires branch September 20, 2024 06:56
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.

Active subscription put "On hold" when first payment expires
2 participants