Open mrudulvajpayee4935 opened 4 weeks ago
@mrudulvajpayee4935 sir I want to work on this issue
@mrudulvajpayee4935 can you please reveiw my PR and please can you help me with its testing how can I test it can you provide me the steps
Hey @kaustubh1106 Thanks for working on this! This will be reviewed soon!
Hi @gorakhnathy7 , I would like to work on this issue. Please assign it to me.
hey i can work on it please assign it to me
Hey @Ankesh2004 @sOnU1002 Thanks for showing interest, But there is another PR in review, which can close this issue. To save everyone's time I would suggested picking another issue.
hi @mrudulvajpayee4935 are there any changes I need to do ?
hey @kaustubh1106 i will review this and let you know.
:memo: Feature Description
Currently, amounts are represented as
i64
values throughout the application. We want to introduce aUnit
struct that explicitly states the denomination. A new type,MinorUnit
, has been added to standardize the flow of amounts across the application. This type will now be used by all the connector flows.Rather than handling conversions in each connector, we will centralize the conversion logic in one place within the core of the application.
:hammer: Possible Implementation
convert
function to receive the amount in their required format.🔖 Note: All the changes needed should be contained within
hyperswitch/crates/router/src/connector/
,crates/router/src/types/api.rs
,crates/router/tests/connectors/
:package: Have you spent some time checking if this feature request has been raised before?
:package: Have you read the Contributing Guidelines?
:package: Submission Process:
Refer here for Terms and conditions for the contest.
:sparkles: Are you willing to submit a PR?