Skip to content

Initial implementation of the FaceTec interop#1

Merged
MOZGIII merged 12 commits intomasterfrom
mozgiii-wip-1
Jun 1, 2021
Merged

Initial implementation of the FaceTec interop#1
MOZGIII merged 12 commits intomasterfrom
mozgiii-wip-1

Conversation

@MOZGIII
Copy link
Contributor

@MOZGIII MOZGIII commented May 11, 2021

The goals of this PR:

  • introduce the API client for the FaceTec SDK
  • implement the robonode server and the logic for interop with the FaceTec Server according to the bioauth v1 plan B architectural diagram
  • provide an example for the error handling patterns and error structure (looking forward to discussing this)
  • provide an example of the code standard for the inline documentation requirements
  • roughly illustrate the required amount of unit tests (though the more - the better)

To do:

dmitrylavrenov
dmitrylavrenov previously approved these changes May 18, 2021
Copy link
Contributor

@dmitrylavrenov dmitrylavrenov left a comment

Choose a reason for hiding this comment

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

Like it

Copy link
Contributor

@dmitrylavrenov dmitrylavrenov left a comment

Choose a reason for hiding this comment

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

The way of comments is clear

@MOZGIII MOZGIII changed the title WIP: Initial implementation of the FaceTec interop Initial implementation of the FaceTec interop Jun 1, 2021
@MOZGIII MOZGIII marked this pull request as draft June 1, 2021 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants