Skip to content

feat(connectors): [Stripe] add extended authorization for cards #9084

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Aug 29, 2025
Merged

feat(connectors): [Stripe] add extended authorization for cards #9084

merged 7 commits into from
Aug 29, 2025

Conversation

AkshayaFoiger
Copy link
Contributor

@AkshayaFoiger AkshayaFoiger commented Aug 29, 2025 •
edited
Loading

Type of Change

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

Description

Extend stripe integration to support extended authorization

How did you test it?

We cannot test this feature until it is enabled by stripe

Sanity Test - cypress ran on the PR

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

@AkshayaFoiger AkshayaFoiger requested review from a team as code owners August 29, 2025 06:24
@semanticdiff-com SemanticDiff.com
Copy link

semanticdiff-com bot commented Aug 29, 2025 •
edited
Loading

Review changes with  SemanticDiff

Changed Files
File Status
  crates/hyperswitch_connectors/src/connectors/stripe/transformers.rs  32% smaller
  crates/hyperswitch_domain_models/src/router_data.rs  0% smaller

@AkshayaFoiger AkshayaFoiger self-assigned this Aug 29, 2025
@AkshayaFoiger AkshayaFoiger changed the title feat(connectors): [Stripe] add extended authorization feat(connectors): [Stripe] add extended authorization for cards Aug 29, 2025
@AkshayaFoiger AkshayaFoiger added the A-connector-integration Area: Connector integration label Aug 29, 2025
hrithikesh026
hrithikesh026 requested changes Aug 29, 2025
View reviewed changes
hrithikesh026
hrithikesh026 approved these changes Aug 29, 2025
View reviewed changes
Copy link
Contributor

@hrithikesh026 hrithikesh026 left a comment

Choose a reason for hiding this comment

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

LGTM

@likhinbopanna likhinbopanna added this pull request to the merge queue Aug 29, 2025
Merged via the queue into main with commit 8cfa966 Aug 29, 2025
25 of 29 checks passed
@likhinbopanna likhinbopanna deleted the stripe-extended-authorization branch August 29, 2025 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@deepanshu-iiitu deepanshu-iiitu deepanshu-iiitu approved these changes

@hrithikesh026 hrithikesh026 hrithikesh026 approved these changes

Assignees

@AkshayaFoiger AkshayaFoiger

Labels
A-connector-integration Area: Connector integration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AkshayaFoiger @deepanshu-iiitu @hrithikesh026 @likhinbopanna