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

feat(wallet/frontend): replace rapyd with gatehub for KYC #1530

Closed
wants to merge 15 commits into from

Conversation

raducristianpopa
Copy link
Member

Context

Closes #1529.

Changes

  • Adds a standalone KYC page since everything will happen in an iframe
  • Removes previous KYC pages (personal/identity verification)
  • Updates user service: remove all obsolete API calls from the frontend
  • Adds getBearerToken method to retrieve the user token for GateHub iframes (should be used for the other iframes as well)

@github-actions github-actions bot added package: wallet/frontend Wallet frontend implementations type: source Source changes labels Sep 4, 2024
type KYCPageProps = InferGetServerSidePropsType<typeof getServerSideProps>

const KYCPage: NextPageWithLayout<KYCPageProps> = ({ url }) => {
// const [openDialog, closeDialog] = useDialog()
Copy link
Contributor

Choose a reason for hiding this comment

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

do we need the comments?

Copy link
Member Author

Choose a reason for hiding this comment

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

Commented out for now. We will keep/remove as needed once we can test the KYC flow.

@Tymmmy
Copy link
Contributor

Tymmmy commented Sep 4, 2024

We need to remove mocks.ts - it is the test image data

@raducristianpopa raducristianpopa force-pushed the rp/1529/remove-rapyd-kyc branch 2 times, most recently from f1a1a91 to 4507237 Compare September 16, 2024 08:04
@github-actions github-actions bot added package: wallet/backend Wallet backend implementations and removed package: wallet/backend Wallet backend implementations labels Sep 16, 2024
@dragosp1011
Copy link
Contributor

it was inherited by #1643 and merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: wallet/frontend Wallet frontend implementations type: source Source changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[GateHub] - Replace Rapyd KYC with GateHub KYC
3 participants