-
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
Conversation
…ms + DB queries + transformers add create collect link flow - validate and initiate flow
refactor(diesel_models/generic_link): add distinct queries as per link_type refactor(collect_link): hide sensitive data feat(collect_link): inject dynamic content in collect link and it's status page
… into payout_link Signed-off-by: Chikke Srujan <[email protected]>
@@ -676,7 +676,13 @@ impl Settings<SecuredSecret> { | |||
.with_list_parse_key("redis.cluster_urls") | |||
.with_list_parse_key("events.kafka.brokers") | |||
.with_list_parse_key("connectors.supported.wallets") | |||
.with_list_parse_key("connector_request_reference_id_config.merchant_ids_send_payment_id_as_connector_request_id"), | |||
.with_list_parse_key("connector_request_reference_id_config.merchant_ids_send_payment_id_as_connector_request_id") |
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.
This is not scalable: this would not allow us to use any suitable key (any possible enum variant) in the HashMap
, but only restricts to the specified keys. We would need to address this in a separate PR.
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.
Looks good for dashboard
Payout { | ||
payout_id: String, | ||
}, |
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.
you might need some changes in the api_events.sql in /crates/analytics/src/docs/scripts/clickhouse
PaymentMethodCollectLink { | ||
link_id: String, | ||
}, |
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.
you might need some changes in the api_events.sql in /crates/analytics/src/docs/scripts/clickhouse
Type of Change
Description
This PR adds below features
Additional Changes
Motivation and Context
How did you test it?
Tested locally
https://galactic-capsule-229427.postman.co/workspace/My-Workspace~2b563e0d-bad3-420f-8c0b-0fd5b278a4fe/collection/9906252-e9eb1c28-489b-4ada-bd00-57c2b4459dea?action=share&creator=9906252
Screencast.from.14-06-24.01.29.08.PM.IST.webm
Checklist
cargo +nightly fmt --all
cargo clippy