Skip to content

[PM-33890] Set up Stripe Subscription Schedule API operations#7289

Merged
amorask-bitwarden merged 1 commit intomainfrom
billing/PM-33890/setup-stripe-subscription-schedules-api-operations
Mar 24, 2026
Merged

[PM-33890] Set up Stripe Subscription Schedule API operations#7289
amorask-bitwarden merged 1 commit intomainfrom
billing/PM-33890/setup-stripe-subscription-schedules-api-operations

Conversation

@amorask-bitwarden
Copy link
Copy Markdown
Contributor

@amorask-bitwarden amorask-bitwarden commented Mar 24, 2026

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-33890

📔 Objective

Registers the pm-32645-defer-price-migration-to-renewal feature flag and adds SubscriptionSchedule CRUD operations (Create, Get, List, Update, Release) to IStripeAdapter / StripeAdapter. Bitwarden has no existing SubscriptionSchedule usage — this establishes the API surface needed for all downstream schedule work in the PM-32645 epic.

No behavioral change — pure infrastructure.

@amorask-bitwarden amorask-bitwarden marked this pull request as ready for review March 24, 2026 15:49
@amorask-bitwarden amorask-bitwarden requested a review from a team as a code owner March 24, 2026 15:49
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

Logo
Checkmarx One – Scan Summary & Detailsd6b8b0fa-9199-453d-807c-0d4e25ffbf44

Great job! No new security vulnerabilities introduced in this pull request

Copy link
Copy Markdown
Collaborator

@sbrown-livefront sbrown-livefront left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@amorask-bitwarden amorask-bitwarden merged commit e7f6d64 into main Mar 24, 2026
42 checks passed
@amorask-bitwarden amorask-bitwarden deleted the billing/PM-33890/setup-stripe-subscription-schedules-api-operations branch March 24, 2026 16:05
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 24, 2026

Codecov Report

❌ Patch coverage is 16.66667% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.84%. Comparing base (7c05036) to head (758fc7f).
⚠️ Report is 68 commits behind head on main.

Files with missing lines Patch % Lines
.../Billing/Services/Implementations/StripeAdapter.cs 16.66% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7289      +/-   ##
==========================================
- Coverage   57.85%   57.84%   -0.01%     
==========================================
  Files        2043     2043              
  Lines       89873    89879       +6     
  Branches     7989     7989              
==========================================
+ Hits        51992    51993       +1     
- Misses      36023    36028       +5     
  Partials     1858     1858              

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@djsmith85 djsmith85 added the t:feature Change Type - Feature Development label Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:feature Change Type - Feature Development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants