Skip to content

feat(connector): [stripebilling] consume necessary fields in stripe billing from webhooks for transaction monitoring support#7722

Merged
Gnanasundari24 merged 56 commits intomainfrom
stripebilling_transaction_monitoring_feilds
May 6, 2025
Merged

feat(connector): [stripebilling] consume necessary fields in stripe billing from webhooks for transaction monitoring support#7722
Gnanasundari24 merged 56 commits intomainfrom
stripebilling_transaction_monitoring_feilds

Conversation

@NISHANTH1221
Copy link
Contributor

@NISHANTH1221 NISHANTH1221 commented Apr 4, 2025

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

Consuming the following fields in the webhook which will support the transaction monitoring feature further.
In stripebilling the subscribed webhook contains an object of that particular invoice. The details needed in the issue are available in the invoice object. Consume Billing address, Retry Count and Next Billing At for the invoice thorugh the webhook and populating in payment intent is the primary goal of this PR.

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

To test this follow the steps, follow the steps in this PR #7461. After doing all the steps you can find the billing address in payment intent table for this corresponding record. Retry count and Next Billing time in feature metadata of the same payment intent

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@NISHANTH1221 NISHANTH1221 self-assigned this Apr 4, 2025
@NISHANTH1221 NISHANTH1221 requested a review from a team as a code owner April 4, 2025 12:14
@semanticdiff-com
Copy link

semanticdiff-com bot commented Apr 4, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  crates/hyperswitch_connectors/src/connectors/stripebilling/transformers.rs  8% smaller

Copy link
Contributor

@srujanchikke srujanchikke left a comment

Choose a reason for hiding this comment

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

We are not storing this data in payment attempt/payment intent, Can you make that changes as well ?

@NISHANTH1221 NISHANTH1221 requested review from a team as code owners April 10, 2025 09:52
@hyperswitch-bot hyperswitch-bot bot added the M-api-contract-changes Metadata: This PR involves API contract changes label Apr 10, 2025
@NISHANTH1221 NISHANTH1221 removed request for a team April 29, 2025 15:22
@NISHANTH1221 NISHANTH1221 added api-v2 S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Apr 30, 2025
Base automatically changed from recurly_invoice_sync_support_with_transaction_monitoring to main May 2, 2025 12:12
@likhinbopanna likhinbopanna dismissed srujanchikke’s stale review May 2, 2025 12:12

The base branch was changed.

@NISHANTH1221 NISHANTH1221 requested a review from srujanchikke May 2, 2025 12:57
@NISHANTH1221 NISHANTH1221 removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label May 4, 2025
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue May 6, 2025
Merged via the queue into main with commit d8bbb80 May 6, 2025
16 of 20 checks passed
@Gnanasundari24 Gnanasundari24 deleted the stripebilling_transaction_monitoring_feilds branch May 6, 2025 07:36
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.

4 participants