-
Notifications
You must be signed in to change notification settings - Fork 4.2k
feat(payments): add support for profile aggregates #5845
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
Review changes with SemanticDiff. Analyzed 7 of 7 files. Overall, the semantic diff is 2% smaller than the GitHub diff.
|
…file-level-payments-aggregate
…/juspay/hyperswitch into profile-level-payments-aggregate
crates/router/src/core/payments.rs
Outdated
time_range: api::TimeRange, | ||
) -> RouterResponse<api::PaymentsAggregateResponse> { | ||
let db = state.store.as_ref(); | ||
let intent_status_with_count = db | ||
.get_intent_status_with_count(merchant.get_id(), &time_range) | ||
.get_intent_status_with_count(merchant.get_id(), profile_id_list.as_ref(), &time_range) |
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.
We can pass the owned value here and avoid clone in the end
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.
Changed to owned type
…/juspay/hyperswitch into profile-level-payments-aggregate
…file-level-payments-aggregate
* 'main' of github.com:juspay/hyperswitch: (51 commits) feat(connector): [DEUTSCHEBANK] Integrate SEPA Payments (#5826) feat(payments_v2): payment intent diesel and domain models changes v2 (#5783) feat(connector): [Fiuu] ADD Wasm Configs (#5874) chore(version): 2024.09.13.0 refactor(core): Update shipping_cost and order_tax_amount to net_amount of payment_attempt (#5844) refactor: return optional request body from build_request_v2 in ConnectorIntegrationV2 trait (#5865) feat(refunds): Refunds aggregate api (#5795) refactor: handle redirections for iframed content (#5591) refactor(payment_links): Update API contract for dynamic transaction details and upgrade UI (#5849) fix(router): add payment_method check in `get_mandate_type` (#5828) fix(connector): [ZSL] compare consr_paid_amt with the total amount for identifying partial payments (#5873) feat(connector): [Novalnet] add Payment flows for cards (#5726) chore(version): 2024.09.12.0 fix(router): return `collect_billing_details_from_wallet_connector` if `always_collect_billing_details_from_wallet_connector ` is false in merchant payment method list (#5854) feat(opensearch): add profile_id and organization_id to /search APIs (#5705) build(deps): bump `sqlx` to `0.8.2` (#5859) refactor: Remove unwanted commented lines (#5851) feat(payments): add support for profile aggregates (#5845) Feat(connector): [Fiuu] Add DuitNow/FPX PaymentMethod (#5841) chore: remove Connectors enum dependency from ConnectorIntegrationV2 trait (#5840) ...
Type of Change
Description
Additional Changes
Motivation and Context
Closes ##5846
How did you test it?
Request :
Response :
Checklist
cargo +nightly fmt --all
cargo clippy