Skip to content

fix(connector): [Peach Payments] fix connector metadata deserialization#9826

Merged
Gnanasundari24 merged 1 commit intomainfrom
fix-peachpayments-metadata
Oct 14, 2025
Merged

fix(connector): [Peach Payments] fix connector metadata deserialization#9826
Gnanasundari24 merged 1 commit intomainfrom
fix-peachpayments-metadata

Conversation

@prasunna09
Copy link
Contributor

@prasunna09 prasunna09 commented Oct 14, 2025

Type of Change

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

Description

fix connector metadata deserialization

Currently we deserialize to camelCase.
but dashboard always sends in snake_case.

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?

Screenshot 2025-10-14 at 2 13 20 PM Screenshot 2025-10-14 at 2 13 38 PM

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 self-assigned this Oct 14, 2025
@prasunna09 prasunna09 requested a review from a team as a code owner October 14, 2025 07:33
@prasunna09 prasunna09 added A-connector-integration Area: Connector integration C-bug Category: Bug labels Oct 14, 2025
@semanticdiff-com
Copy link

semanticdiff-com bot commented Oct 14, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  crates/hyperswitch_connectors/src/connectors/peachpayments/transformers.rs  0% smaller

@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Oct 14, 2025
Merged via the queue into main with commit 859b3b1 Oct 14, 2025
27 of 32 checks passed
@Gnanasundari24 Gnanasundari24 deleted the fix-peachpayments-metadata branch October 14, 2025 10:15
drdholu pushed a commit to drdholu/hyperswitch that referenced this pull request Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-connector-integration Area: Connector integration C-bug Category: Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants