Skip to content

Conversation

@shahzaibj
Copy link
Contributor

@shahzaibj shahzaibj commented Aug 26, 2025

@shahzaibj shahzaibj self-assigned this Aug 26, 2025
@github-actions
Copy link

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

Click here to Learn more.

@github-actions github-actions bot added the msal label Aug 26, 2025
@shahzaibj shahzaibj changed the title Shahzaib/duna fixes Fix for App Link Usage in DUNA / SSO scenarios Aug 26, 2025
@shahzaibj shahzaibj marked this pull request as ready for review August 26, 2025 23:35
Copilot AI review requested due to automatic review settings August 26, 2025 23:35
@shahzaibj shahzaibj requested review from a team as code owners August 26, 2025 23:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR addresses App Link usage in DUNA (Device-based Unified Authentication) and SSO scenarios by migrating from deprecated LocalBroadcastManager to standard Android broadcast mechanisms and updating dependencies.

  • Replaces LocalBroadcastManager with standard Android broadcast receivers for improved compatibility
  • Updates browser library dependency from version 1.0.0 to 1.7.0
  • Adds proper receiver registration handling for Android 13+ security requirements

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
CurrentTaskBrowserTabActivity.java Migrates from LocalBroadcastManager to standard broadcasts with proper Android 13+ security handling
gradle/versions.gradle Updates browser library dependency to version 1.7.0
common Updates submodule commit reference
changelog Documents the fix in release notes

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@shahzaibj shahzaibj merged commit b7c6243 into dev Aug 28, 2025
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants