Skip to content

[MS-710] Add UNCOOPERATIVE_CHILD exit option#895

Merged
meladRaouf merged 1 commit into
mainfrom
ms-710-UncooperativeChild
Sep 12, 2024
Merged

[MS-710] Add UNCOOPERATIVE_CHILD exit option#895
meladRaouf merged 1 commit into
mainfrom
ms-710-UncooperativeChild

Conversation

@meladRaouf
Copy link
Copy Markdown
Collaborator

No description provided.

@cla-bot cla-bot Bot added the ... label Sep 4, 2024
@meladRaouf meladRaouf force-pushed the ms-710-UncooperativeChild branch from 2cb9c98 to cf471dc Compare September 5, 2024 00:04
@meladRaouf meladRaouf requested review from a team, BurningAXE, TristramN, alex-vt, alexandr-simprints, luhmirin-s and ybourgery and removed request for a team September 5, 2024 00:12
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 5, 2024

Copy link
Copy Markdown
Contributor

@luhmirin-s luhmirin-s left a comment

Choose a reason for hiding this comment

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

Would it be an overkill to only show the option if child biometrics are being collected in the project?

@meladRaouf
Copy link
Copy Markdown
Collaborator Author

meladRaouf commented Sep 5, 2024

Would it be an overkill to only show the option if child biometrics are being collected in the project?

@luhmirin-s I think this is a great idea.
@jonsimprints, what are your thoughts? Should we display this exit option only when child bio is active?

@jonsimprints
Copy link
Copy Markdown
Collaborator

@meladRaouf how would we objectively define a child?

@meladRaouf
Copy link
Copy Markdown
Collaborator Author

@meladRaouf how would we objectively define a child?

Good question,
If any SDK is enabled and the age limit is below, for example, 6 years, we can conclude that this SDK supports child bio.

@meladRaouf meladRaouf merged commit 00ec9e5 into main Sep 12, 2024
@meladRaouf meladRaouf deleted the ms-710-UncooperativeChild branch September 12, 2024 09:27
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