-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
refactor: separate DB queries and HTML creation for payout links #4967
Merged
Merged
Changes from 3 commits
Commits
Show all changes
101 commits
Select commit
Hold shift + click to select a range
b23edb3
feat(collect_link): basic setup for collect_links - app structs / enu…
kashif-m cc48b06
wip: feat(collect_link): add render pm collect link flow
kashif-m dcfa4bb
feat(collect_link): add basic initiate + status + expiry templates
kashif-m 58e5e78
feat(collect_link): store pm_collect_link_config in merchant account
kashif-m 89277ce
refactor(collect_link): cargo hack fixes
kashif-m 843c2c8
feat(collect_link): inject enabled_payment_methods
kashif-m a2daf17
Merge remote-tracking branch 'origin/main' into pm_collect_link
kashif-m 97b40ed
refactor(collect_link): update styles
kashif-m 7cc75ac
Merge remote-tracking branch 'origin/main' into pm_collect_link
kashif-m 7057338
feat(collect_link): add widget options
kashif-m 5bafcd1
feat(payment_methods): enable client secret auth for CREATE payment_m…
kashif-m e0f7fac
feat(collect_link): read enabled_payment_methods in /admin and collec…
kashif-m f02b526
Merge remote-tracking branch 'origin/main' into pm_collect_link
kashif-m 6ce91c5
Merge remote-tracking branch 'origin/main' into pm_collect_link
kashif-m 3913d46
add payout link support
srujanchikke 84ab2d9
add confirm api
srujanchikke 7411a56
remove comments
srujanchikke 3bdffff
Merge remote-tracking branch 'origin/main' into pm_collect_link
kashif-m 85da820
add slash to payout_link scope
51e7ad7
Merge branch 'pm_collect_link' of https://github.com/juspay/hyperswit…
srujanchikke acea448
Merge branch 'payout_link' of https://github.com/juspay/hyperswitch i…
srujanchikke ab5d140
resolve merge conflicts
srujanchikke 8d1e34e
chore: run formatter
hyperswitch-bot[bot] f7a341a
Merge remote-tracking branch 'origin/main' into pm_collect_link
kashif-m 26c9ecf
remove schema.rs
srujanchikke 1dee131
Merge branch 'payout_link' of https://github.com/juspay/hyperswitch i…
srujanchikke 38f94e8
resolve schema.rs file conflict
srujanchikke cf25247
refactor: add openapi changes
kashif-m 7cec5c0
chore: run formatter
hyperswitch-bot[bot] f6cfde2
merge branch
srujanchikke 3c9a3a2
clippy fix
srujanchikke 6be0325
fix validate openapi spec
srujanchikke 85e64dd
clippy fix
srujanchikke 3791ebb
chore: run formatter
hyperswitch-bot[bot] cc0b09d
fix schema
srujanchikke 762a24b
refactor: separate DB queries and HTML creation for payout links
kashif-m 691082e
add enabled payout method in payout_link
srujanchikke 52d8dd8
fix enabled payment methods
srujanchikke acbea09
make enabled payment methods optional for payout links
srujanchikke 8b28952
chore: run formatter
hyperswitch-bot[bot] f7e52ea
open api check fix
srujanchikke 9c9a3c6
open api fix
srujanchikke e5dc100
refactor: separate queries for payout links
kashif-m 02c2763
openapi fix
srujanchikke f5b7cbf
ci check fix
srujanchikke dcb32db
chore: run formatter
hyperswitch-bot[bot] 5d09e85
feat(payout_link): add expired and status pages
kashif-m 073a5ce
Merge remote-tracking branch 'origin/main' into payout_link_ref
kashif-m d1507c7
ci check fix
srujanchikke ead64a9
Merge branch 'payout_link' of github.com:juspay/hyperswitch into payo…
srujanchikke 6e0d116
chore: run formatter
hyperswitch-bot[bot] 13e011b
minor refactor
srujanchikke b616451
Merge branch 'payout_link' of github.com:juspay/hyperswitch into payo…
srujanchikke cb10520
refactor: cargo fix
kashif-m eef38fa
Merge remote-tracking branch 'origin/payout_link' into payout_link_ref
kashif-m bc1dd6d
chore: run formatter
hyperswitch-bot[bot] 571824b
refactor: re-generate openAPI spec
kashif-m 3d9f1ad
Merge branch 'main' into pm_collect_link
kashif-m 1d8af46
Merge branch 'pm_collect_link' into payout_link
kashif-m fd7c412
Merge remote-tracking branch 'origin/payout_link' into payout_link_ref
kashif-m 4e06034
resolve comments
srujanchikke b218946
chore: run formatter
hyperswitch-bot[bot] 0b0dfd1
refactor(generic_links): move payout_link_config to business_profile
kashif-m aabf12b
refactor: inject flow type for pm_collect_link
kashif-m d668130
docs(openapi): re-generate OpenAPI specification
hyperswitch-bot[bot] 4e5ac76
resolve comments
a41ee95
remove unnecessary logs
026ad06
chore: run formatter
hyperswitch-bot[bot] b9dcceb
clippy fix
00bc65e
refactor(payout_links): map domain name to the payout link and add a …
kashif-m 2ee89eb
Merge remote-tracking branch 'origin/main' into payout_link_ref
kashif-m a939970
refactor: missing import
kashif-m b889d33
refactor: clippy fix
kashif-m 9c7258e
update payouts in confirm
982d4b5
chore: run formatter
hyperswitch-bot[bot] 0eb0136
update payout_type in PayoutUpdate
e958839
refactor: revert client_secret auth in POST /payment_methods
kashif-m de2434a
Merge remote-tracking branch 'origin/main' into pm_collect_link
kashif-m ba9880d
refactor: missing import
kashif-m f3f1bfc
Merge remote-tracking branch 'origin/pm_collect_link' into payout_link
kashif-m a285679
Merge remote-tracking branch 'origin/payout_link' into payout_link_ref
kashif-m 2c62a96
refactor: clippy fixes
kashif-m 4717b07
Merge remote-tracking branch 'origin/pm_collect_link' into payout_link
kashif-m 8bc98ef
refactor: revert payouts insert query
kashif-m e3da8fc
refactor: add default env configs in docker_compose.toml and update m…
kashif-m ce0f155
feat: attach payout_id to Payout events
kashif-m 4c5496b
Merge remote-tracking branch 'origin/main' into pm_collect_link
kashif-m 1797a83
Merge remote-tracking branch 'origin/main' into payout_link_ref
kashif-m 6a47a27
clippy fix
d0543e7
Merge branch 'pm_collect_link' into payout_link
srujanchikke d301039
refactor: resolve comments
kashif-m e62b354
Merge remote-tracking branch 'origin/payout_link' into payout_link_ref
kashif-m 1602734
refactor: clippy fixes
kashif-m baca310
refactor: clippy fixes
kashif-m 21ebf13
refactor: resolve comments
kashif-m 5a65b3d
refactor: migration update
kashif-m 6c35e7e
refactor: add comments
kashif-m cedb025
chore: missing validations
kashif-m 91c269b
refactor: update generic link_status
kashif-m 6a34bb7
chore: comments
kashif-m d99c3f7
chore: updated env configs
kashif-m File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -952,7 +952,7 @@ pub struct PaymentMethodCollectLinkRequest { | |
|
||
/// The unique identifier of the customer. | ||
#[schema(value_type = String, example = "cus_92dnwed8s32bV9D8Snbiasd8v")] | ||
pub customer_id: String, | ||
pub customer_id: id_type::CustomerId, | ||
|
||
#[serde(flatten)] | ||
pub ui_config: Option<api_enums::CollectLinkConfig>, | ||
|
@@ -979,7 +979,7 @@ pub struct PaymentMethodCollectLinkResponse { | |
|
||
/// The unique identifier of the customer. | ||
#[schema(value_type = String, example = "cus_92dnwed8s32bV9D8Snbiasd8v")] | ||
pub customer_id: String, | ||
pub customer_id: id_type::CustomerId, | ||
|
||
/// Time when this link will be expired in ISO8601 format | ||
#[schema(value_type = PrimitiveDateTime, example = "2025-01-18T11:04:09.922Z")] | ||
|
@@ -1019,7 +1019,7 @@ pub struct PaymentMethodCollectLinkDetails { | |
pub pub_key: masking::Secret<String>, | ||
pub client_secret: masking::Secret<String>, | ||
pub pm_collect_link_id: String, | ||
pub customer_id: String, | ||
pub customer_id: id_type::CustomerId, | ||
#[serde(with = "common_utils::custom_serde::iso8601")] | ||
pub session_expiry: time::PrimitiveDateTime, | ||
pub return_url: Option<String>, | ||
|
@@ -1031,7 +1031,7 @@ pub struct PaymentMethodCollectLinkDetails { | |
#[derive(Clone, Debug, serde::Serialize)] | ||
pub struct PaymentMethodCollectLinkStatusDetails { | ||
pub pm_collect_link_id: String, | ||
pub customer_id: String, | ||
pub customer_id: id_type::CustomerId, | ||
#[serde(with = "common_utils::custom_serde::iso8601")] | ||
pub session_expiry: time::PrimitiveDateTime, | ||
pub return_url: Option<String>, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same here. |
||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Same here, this can be a
url::Url
.