Skip to content

Conversation

prasunna09
Copy link
Contributor

@prasunna09 prasunna09 commented Jun 10, 2025

Type of Change

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

Description

removed the refunds_v2 feature flag from router, payment method, domain model, diesel model, api model and storage impl crate.

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?

Sanity testing for v1 and v2 - refunds flow

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

@prasunna09 prasunna09 requested review from a team as code owners June 10, 2025 14:52
Base automatically changed from remove-payment-methods-v2-feature-flag to main June 11, 2025 19:09
SanchithHegde
SanchithHegde previously approved these changes Jun 12, 2025
@SanchithHegde SanchithHegde removed request for a team June 12, 2025 13:43
jarnura
jarnura previously approved these changes Jun 17, 2025
SanchithHegde
SanchithHegde previously approved these changes Jun 20, 2025
@likhinbopanna likhinbopanna added this pull request to the merge queue Jun 25, 2025
Merged via the queue into main with commit c5c0e67 Jun 25, 2025
21 of 24 checks passed
@likhinbopanna likhinbopanna deleted the remove-refunds_v2-feature-flag branch June 25, 2025 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-refactor Category: Refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants