Skip to content

build: DCMAW-17339 Bump ID Check SDK version to 0.37.4#787

Merged
de1phine merged 2 commits into
mainfrom
feat/DCMAW-17339-abort-screens-close-button
Apr 23, 2026
Merged

build: DCMAW-17339 Bump ID Check SDK version to 0.37.4#787
de1phine merged 2 commits into
mainfrom
feat/DCMAW-17339-abort-screens-close-button

Conversation

@de1phine
Copy link
Copy Markdown
Contributor

@de1phine de1phine commented Apr 22, 2026

DCMAW-17339: Update ID Check SDK version to 0.37.4

  • Updated ID Check SDK from 0.37.0 to 0.37.4 to get the new ConfirmAnotherWayHostDialogFragment which adds a close button to the confirm your identity another way flow screens
  • Added launchSingleTop = true to the AllowModalToShow navigation in ProveYourIdentityRoot
    • This resolves a bug when the device is rotated on the first abort screen inside the ID Check SDK, the activity recreation causes AllowModalToShow to re-trigger, which was adding a duplicate /modal onto the navigation back stack. This results in the user needing to press the X button twice to close the second abort screen (AbortedReturnToDesktopWeb).

Evidence of the change

DAD-Light-Theme.webm
MAM-Dark-Theme.webm

Checklist

  • Check against acceptance criteria
  • Add automated tests
  • Self-review code

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 22, 2026

Analytics test report is available to download.

@de1phine de1phine force-pushed the feat/DCMAW-17339-abort-screens-close-button branch from 77ec3b7 to 9c19a58 Compare April 23, 2026 08:30
@de1phine de1phine marked this pull request as ready for review April 23, 2026 08:30
@de1phine de1phine requested review from a team as code owners April 23, 2026 08:30
@sonarqubecloud
Copy link
Copy Markdown

@de1phine de1phine merged commit 542eb88 into main Apr 23, 2026
18 checks passed
@de1phine de1phine deleted the feat/DCMAW-17339-abort-screens-close-button branch April 23, 2026 08:57
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