Skip to content

fix(billing): require card-backed trial again#56

Merged
paulocastellano merged 5 commits into
mainfrom
fix/require-card-for-trial
May 21, 2026
Merged

fix(billing): require card-backed trial again#56
paulocastellano merged 5 commits into
mainfrom
fix/require-card-for-trial

Conversation

@paulocastellano
Copy link
Copy Markdown
Contributor

Summary

  • revert the no-card signup trial behavior so new accounts no longer receive automatic generic trial access
  • require checkout-based Stripe subscription trial again by restoring trialDays on subscribe and ->trialDays(...) in billing checkout
  • align account trial logic and feature tests with card-backed trial semantics (subscription trial only)

Test plan

  • vendor/bin/pint --dirty --format agent
  • php artisan test --compact tests/Feature/Auth/TrialOnSignupTest.php tests/Feature/Middleware/TrialMiddlewareAccessTest.php tests/Feature/BillingControllerTest.php tests/Unit/Models/AccountTest.php

Made with Cursor

paulocastellano and others added 5 commits May 21, 2026 09:49
Revert the no-card signup trial flow so access depends on a Stripe subscription trial started at checkout, preventing app access before a payment method is collected.

Co-authored-by: Cursor <cursoragent@cursor.com>
Show the checkout-based trial duration again on the subscribe page and rename the signup billing test file so it matches the current no-trial-at-signup behavior.

Co-authored-by: Cursor <cursoragent@cursor.com>
Add a trypost config toggle to switch between card-required checkout trials and no-card signup trials, and wire signup, checkout, access gating, UI copy, and tests to both modes.

Co-authored-by: Cursor <cursoragent@cursor.com>
Keep the trial card requirement switch as a plain boolean in config/trypost.php so changing the app flow is a single in-repo config edit without env wiring.

Co-authored-by: Cursor <cursoragent@cursor.com>
Increase the default trial duration from 7 to 8 days in the cashier configuration to better align with user feedback and improve trial experience.
@paulocastellano paulocastellano merged commit c18c1b9 into main May 21, 2026
2 checks passed
@paulocastellano paulocastellano deleted the fix/require-card-for-trial branch May 21, 2026 23:12
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.

1 participant