Skip to content

Conversation

swangi-kumari
Copy link
Contributor

Type of Change

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

Description

fix amount conversion framework for Rapyd
According to doc - https://docs.rapyd.net/en/create-payment.html , amount should be of FloatType (BaseUnit).

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?

We can not test it, as we dont have credentials for Rapyd connector

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

@swangi-kumari swangi-kumari added A-connector-integration Area: Connector integration C-refactor Category: Refactor labels Mar 9, 2025
@swangi-kumari swangi-kumari self-assigned this Mar 9, 2025
@swangi-kumari swangi-kumari requested a review from a team as a code owner March 9, 2025 19:10
Copy link

semanticdiff-com bot commented Mar 9, 2025

Review changes with  SemanticDiff

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

@likhinbopanna likhinbopanna added this pull request to the merge queue Mar 10, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 10, 2025
@SanchithHegde SanchithHegde added this pull request to the merge queue Mar 10, 2025
Merged via the queue into main with commit 8f1bab6 Mar 10, 2025
15 of 20 checks passed
@SanchithHegde SanchithHegde deleted the rapyd-amount branch March 10, 2025 17:54
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-refactor Category: Refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants