Skip to content

[MS-1243] Controlling the matcher execution with the isMatcherRunning flag#1462

Merged
alexandr-simprints merged 1 commit into
release/2025.4.0from
MS-1243-commcare-sid-restarts-identification-after-screen-lock-during-1-n-identification
Nov 10, 2025
Merged

[MS-1243] Controlling the matcher execution with the isMatcherRunning flag#1462
alexandr-simprints merged 1 commit into
release/2025.4.0from
MS-1243-commcare-sid-restarts-identification-after-screen-lock-during-1-n-identification

Conversation

@alexandr-simprints
Copy link
Copy Markdown
Contributor

JIRA ticket
Will be released in: 2025.4.0

Root cause analysis (for bugfixes only)

First known affected version: 2025.4.0

When performing a 1:N identification with commcare around 15k records, SID restarts the identification process if the device screen locks during the operation. This causes the ongoing matching session to be lost and forces the user to restart the entire identification process.

Notable changes

isMatcherRunning is added to the MatchViewModel that prevents ViewModel to restart the ongoing matcher

Testing guidance

  • Start a 1:N identification with ~15,000 records.
  • Allow the screen to lock automatically or lock it manually during the process.
  • Unlock the screen and verify that matcher is still running

Additional work checklist

  • Effect on other features and security has been considered
  • Design document marked as "In development" (if applicable)
  • External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
  • Test cases in Testiny are up to date (or ticket created)
  • Other teams notified about the changes (if applicable)

@sonarqubecloud
Copy link
Copy Markdown

@alexandr-simprints alexandr-simprints merged commit dc62511 into release/2025.4.0 Nov 10, 2025
13 checks passed
@alexandr-simprints alexandr-simprints deleted the MS-1243-commcare-sid-restarts-identification-after-screen-lock-during-1-n-identification branch November 10, 2025 19:06
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