-
Notifications
You must be signed in to change notification settings - Fork 2.2k
refactor(refunds): set Instant as the default RefundType #7824
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.
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
Conversation
Changed Files
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than that, looks good to me!
@@ -24,8 +24,8 @@ use crate::enums::{Currency, RefundStatus}; | |||
#[serde(rename_all = "snake_case")] | |||
#[strum(serialize_all = "snake_case")] | |||
pub enum RefundType { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ShivanshMathurJuspay Why do we have this enum in the analytics
crate as well? And does it need to derive Default
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// TODO RefundType api_models_oss need to mapped to storage_model
This was added when we had added refund analytics back in 2023
Type of Change
Description
Additional Changes
Motivation and Context
How did you test it?
Response
Response
Refund will go to succeeded, as now we have set default refund_type as Instant.
Before this change all the refund was being Scheduled, if we don't pass
"refund_type": "instant"
In db we can see
refund_type | instant_refund
Checklist
cargo +nightly fmt --all
cargo clippy