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

refactor repo · new ux · prove circuits · new sdk #177

Merged
merged 130 commits into from
Aug 29, 2024

Conversation

remicolin
Copy link
Member

@remicolin remicolin commented Aug 17, 2024

circuits

  • rename circom circuits according to this new pattern: sha256WithRSAEncryption_65537 -> rsa_65537_sha256
  • add prove circuit for 1Step flow
  • refactor
  • new scripts used by sdk to download circuits from aws and compile them

common

  • refactor
  • generate passport data dynamically
  • add mock certificates for ecdsa

app

  • new UI/UX based on QR code scanning
  • add screen to generate mockpassport data with different sig/hash
  • download zkey dynamically according to passport data
  • improvement of overall UX: more toast, add progress bar if circuit is not downloaded yet, etc.
  • clean/refactor

sdk

  • implement dynamic selection of the verifier according to response form app
  • implement QR code generator with native web-socket connection and led component to display connection states
  • fix/clean/refactor
  • add yarn command
  • implement prettierrc

@remicolin remicolin changed the title New ux refactor circuits rename and refactor circuits Aug 17, 2024
@remicolin remicolin marked this pull request as ready for review August 29, 2024 07:17
@remicolin remicolin changed the title rename and refactor circuits refactor repo · new ux · prove circuits · new sdk Aug 29, 2024
@remicolin remicolin merged commit 40ed5c9 into dev Aug 29, 2024
1 check passed
@remicolin remicolin deleted the new-ux-refactor-circuits branch August 29, 2024 09:53
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.

2 participants