refactor: handle redirections for iframed content #5591
Merged
Conversation
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
(cherry picked from commit db6df219a9cd12d619268f388b26c88692d858f8)
hyperswitch-bot
bot
added
the
M-api-contract-changes
2 tasks
kashif-m
added
A-core
|
Remove console log after figuring out the log pipeline |
jarnura
previously approved these changes
Sep 11, 2024
Sakilmostak
approved these changes
Sep 11, 2024
Removed from payment links, will remove from payouts (in separate PR) once the log pipelines are in place |
jarnura
approved these changes
Sep 11, 2024
pixincreate
added a commit
that referenced
this pull request
Sep 13, 2024
* '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) ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
(cherry picked from commit db6df219a9cd12d619268f388b26c88692d858f8)
Type of Change
Description
Additional Changes
How did you test it?
[Payments] open links (no auth redirections)
Screen.Recording.2024-08-12.at.1.53.41.PM.mov
[Payments] open links (with auth redirections)
Screen.Recording.2024-08-12.at.1.54.19.PM.mov
[Payments] secure links (no auth redirections)
Screen.Recording.2024-08-12.at.1.52.21.PM.mov
[Payments] secure links (with auth redirections)
Screen.Recording.2024-08-12.at.1.49.31.PM.mov
[Payouts] secure links (with status page redirection)
Screen.Recording.2024-08-12.at.1.55.23.PM.mov
Checklist
cargo +nightly fmt --allcargo clippy