Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Components] Add id capture screen #177

Merged
merged 51 commits into from
Jan 23, 2024
Merged

[Components] Add id capture screen #177

merged 51 commits into from
Jan 23, 2024

Conversation

ayinloya
Copy link
Collaborator

@ayinloya ayinloya commented Jan 19, 2024

This adds instruction screen for document capture.

Install dependencies localling npm install
Run npm run storybook
Open http://localhost:6006/ to few the stories
Under the id-capture

  • verify that id capture page is rendered correctly when pending camera permission
  • verify that id capture page is rendered correctly when permission is granted

Depends on #176

Handle back navigation in back of id
Add text button to review
Update image height in review
Fix DocumentInstruction story
Update video preview to capture more context
Add styles and typography
Copy link

This branch has been deployed to s3 / cloudfront.

✅ Preview URL for Smart Camera Web:
https://cdn.smileidentity.com/js/preview-add-id-capture-screen/smart-camera-web.js
✅ Preview URL for Embed:
https://cdn.smileidentity.com/inline/preview-add-id-capture-screen/js/script.min.js

Copy link

@ayinloya ayinloya requested a review from Yeboahmedia January 19, 2024 12:10
Base automatically changed from add-document-capture-instructions to main January 23, 2024 11:48
Copy link
Contributor

@mldangelo mldangelo left a comment

Choose a reason for hiding this comment

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

Thank you @ayinloya

@ayinloya ayinloya merged commit 14a3563 into main Jan 23, 2024
7 checks passed
@ayinloya ayinloya deleted the add-id-capture-screen branch January 23, 2024 18:04
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.

None yet

3 participants