Skip to content

[MS-1605] Add face capture attempt events - #1853

Merged
meladRaouf merged 1 commit into
mainfrom
feature/face-capture-attempet-event
Oct 15, 2026
Merged

meladRaouf merged 1 commit into
mainfrom
feature/face-capture-attempet-event

Conversation

@meladRaouf

Copy link
Copy Markdown
Collaborator

MS-1605
Will be released in: 2027.1.0

Notable changes

Adds a FaceCaptureAttemptEvent emitted once per face auto-capture attempt.

Design doc: [link]

Testing guidance

  • enable face auto-capture, run an enrolment, and inspect the session upload — expect exactly one FaceCaptureAttempt event per attempt, with timeToFirstValidMs matching the attempt start, and run durations summing to endTime − startTime.
  • Abandonment: press capture, stay too far so no valid frame is found, back out of the screen — one event with validFrameCount: 0 and no timeToFirstValidMs.
  • Manual capture mode: no FaceCaptureAttempt event is emitted.
  • Backend ingestion is not yet available; the event will be rejected on upload until Cloud adds the schema.

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) — Cloud ticket for ingestion

@cla-bot cla-bot Bot added the ... label Oct 6, 2026
@meladRaouf
meladRaouf requested review from BurningAXE, alex-vt, alexandr-simprints and luhmirin-s and removed request for luhmirin-s October 6, 2026 10:45
@meladRaouf
meladRaouf force-pushed the feature/face-capture-attempet-event branch from 354e4b6 to 01c7b6d Compare October 6, 2026 14:33
@meladRaouf

Copy link
Copy Markdown
Collaborator Author

@copilot resolve the merge conflicts in this pull request

@cla-bot

cla-bot Bot commented Oct 15, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement. For us to review and merge your code, don't hesitate to get in touch with @chrisjroyce, @TristramN or @mharvey83 to get yourself added.

@cla-bot cla-bot Bot removed the ... label Oct 15, 2026

Copilot AI commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Merged main and resolved the conflict in LiveFeedbackViewModel.kt, preserving the face-attempt cleanup and incorporating main’s changes. Merge commit: 8af9987.

@meladRaouf
meladRaouf force-pushed the feature/face-capture-attempet-event branch from 8af9987 to 70352b9 Compare October 15, 2026 08:50
@cla-bot cla-bot Bot added the ... label Oct 15, 2026
@luhmirin-s
luhmirin-s self-requested a review October 15, 2026 08:53
@sonarqubecloud

Copy link
Copy Markdown

@meladRaouf
meladRaouf merged commit e39dad1 into main Oct 15, 2026
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.

3 participants