Skip to content

DUNA flow: Resume Auth , Fixes AB#3079802#2557

Merged
p3dr0rv merged 102 commits intodevfrom
pedroro/switch_to_webview
Mar 19, 2025
Merged

DUNA flow: Resume Auth , Fixes AB#3079802#2557
p3dr0rv merged 102 commits intodevfrom
pedroro/switch_to_webview

Conversation

@p3dr0rv
Copy link
Copy Markdown
Contributor

@p3dr0rv p3dr0rv commented Dec 11, 2024

This PR addresses step 16 of the Switch browser flow

  • BrokerBrowserRedirectActivity will re-start the WebViewAuthorizationFragment so we update the intent data and handle the activity being resumed on WebViewAuthorizationFragment.onResume

  • The switch browser resume endpoint requires the client ID, so we need to pass this value at the moment we start the WebAuthorizaFragment.

  • Refactor AuthorizationActivityFactory to work with one data parameter to create the auth intent.

AB#3079802

@github-actions
Copy link
Copy Markdown

❌ Work item link check failed. Description does not contain AB#{ID}.

Click here to Learn more.

@p3dr0rv p3dr0rv changed the title [WIP] Back to WebView DUNA flow: Resume Auth Dec 12, 2024
@p3dr0rv p3dr0rv marked this pull request as ready for review December 12, 2024 18:57
@p3dr0rv p3dr0rv requested a review from a team as a code owner December 12, 2024 18:57
Copy link
Copy Markdown
Contributor

@mohitc1 mohitc1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants