Skip to content

Add KYC Service and enable getting user's persona accountId#239

Merged
ocasta181 merged 12 commits intodevelopfrom
task/andrin/str-679
Jul 28, 2023
Merged

Add KYC Service and enable getting user's persona accountId#239
ocasta181 merged 12 commits intodevelopfrom
task/andrin/str-679

Conversation

@ocasta181
Copy link
Contributor

@ocasta181 ocasta181 commented Jul 26, 2023

STR-679

Changes:

  • Add an endpoint to /users/ that returns the user's persona accountId. It either returns a previously existing persona accountId or creates a new one and returns that.
  • Add KYC service for identifying if the user meets the KYC level needed for a transaction

To test:

  • Use Create Identity Table migrations#18
  • Create a user and hit the /users/persona-account-id endpoint. Confirm a new persona account has been created in the Persona dashboard.
  • Run the unit test TestMeetsRequirements() in kyc_tests.go

@ocasta181 ocasta181 changed the title Enable getting user's persona accountId Add KYC Service and enable getting user's persona accountId Jul 26, 2023
akfoster and others added 3 commits July 26, 2023 18:46
Co-authored-by: saito-sv <7920256+saito-sv@users.noreply.github.com>
@ocasta181 ocasta181 marked this pull request as ready for review July 27, 2023 06:10
auroter
auroter previously approved these changes Jul 27, 2023
Copy link
Contributor

@auroter auroter left a comment

Choose a reason for hiding this comment

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

image

frostbournesb
frostbournesb previously approved these changes Jul 27, 2023
saito-sv
saito-sv previously approved these changes Jul 27, 2023
@ocasta181 ocasta181 dismissed stale reviews from saito-sv and frostbournesb via 839065b July 28, 2023 20:05
@ocasta181 ocasta181 merged commit 09a2ebe into develop Jul 28, 2023
@ocasta181 ocasta181 deleted the task/andrin/str-679 branch July 28, 2023 20:21
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.

4 participants