-
Notifications
You must be signed in to change notification settings - Fork 1
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
Request KYC endpoint #191
Request KYC endpoint #191
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There're more places where we need to handle the new state.
In fplus-lib/src/core/mod.rs
, line 835, in complete_governance_review
, there's a whitelist of states for which complete_governance_review
can be triggered. Right now that's Submitted
, AdditionalInfoRequired
and AdditionalInfoSubmitted
. We should add the new KYCRequested
to it, so that allocator can override it if needed.
In fplus-lib/src/core/mod.rs
, line 1711, we're validating the flow (read-only endpoint it seems) (part of GH Action: https://github.com/fidlabs/allocator-template/blob/main/.github/workflows/flow-validator.yml ). We should probably consider it invalid if we're in KYCRequested, but we do have current allocations.
Superseded by #195 |
Endpoint: verifier/application/request_kyc
Description:
The functionality enables Allocators to request their clients to undergo the KYC process.
The
verifier/application/request_kyc
endpoint checks, based on the data received (KYCRequestedInfo
), the current state of the application. If the condition is met, it:kyc requested
.Usage Condition:
To execute this endpoint, the following conditions must be met:
Submitted
state.Deployment Considerations:
KYC_URL
- variable should be set to the URL of the KYC portalExample KYCRequestedInfo: