Skip to content

Conversation

@p3dr0rv
Copy link
Contributor

@p3dr0rv p3dr0rv commented Apr 25, 2025

  • Move IBrokerRestrictionsManager.kt from broker to common
  • Add SdmQrPinManager
  • Move camera consent code from WebViewAuthorizationFragment.java to CameraPermissionRequestHandler.kt
  • Camera logic change: previously we show camera QR PIN consent dialog if request origin is from MS, otherwise we use default behavior. Now we use the SdmQrPinManager to determine if we should show the camera QR PIN consent dialog. (see: CameraPermissionRequestHandler.handle(request: PermissionRequest, context: Context)

Related PR: https://github.com/AzureAD/ad-accounts-for-android/pull/3089

Run manual UI test for validation https://identitydivision.visualstudio.com/Engineering/_testPlans/execute?planId=2007357&suiteId=2810290

AB#3121976

@p3dr0rv p3dr0rv requested a review from a team as a code owner April 25, 2025 03:52
@github-actions
Copy link

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

Click here to Learn more.

@github-actions
Copy link

✅ Work item link check complete. Description contains link AB#3121976 to an Azure Boards work item.

@github-actions github-actions bot changed the title Move camera logic to CameraPermissionRequestHandler and add SdmQrPinManager Move camera logic to CameraPermissionRequestHandler and add SdmQrPinManager , Fixes AB#3121976 Apr 25, 2025
Copy link
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.

Left some suggestions

@p3dr0rv p3dr0rv added the Skip-Consumers-Check Only include this if making a breaking change purposefully, and there is an MSAL/ADAL/Broker PR label Apr 29, 2025
@p3dr0rv p3dr0rv requested a review from a team as a code owner April 29, 2025 22:48
@p3dr0rv p3dr0rv merged commit c1af230 into dev Apr 30, 2025
22 of 25 checks passed
@p3dr0rv p3dr0rv deleted the pedroro/qr-pin-rework branch April 30, 2025 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip-Consumers-Check Only include this if making a breaking change purposefully, and there is an MSAL/ADAL/Broker PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants