Refactor ComPilot providers to separate KYC and Signature Gating workflows - #452
Open
grmkris wants to merge 1 commit into
Open
Refactor ComPilot providers to separate KYC and Signature Gating workflows#452grmkris wants to merge 1 commit into
grmkris wants to merge 1 commit into
Conversation
…flows - Created separate challenge functions for KYC and Signature Gating in compilot-config.ts - Added KYCComPilotProvider and SignatureGatingComPilotProvider components - Updated _app.tsx to use both providers with proper nesting - Modified challenge API endpoint to handle workflowType parameter - This ensures correct workflow IDs are used consistently throughout the system 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
grmkris
requested review from
TilenOman50,
acuderman,
bogdancirlugea2,
eveelin,
joelamouche,
kevin-pannetier,
markoftw and
thierry-nexera
as code owners
June 16, 2025 10:54
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
Created separate challenge functions in
compilot-config.ts:generateChallengeKYC- addsworkflowType: "kyc"to requestsgenerateChallengeSignatureGating- addsworkflowType: "signatureGating"to requestsAdded provider components in
src/providers/ComPilotProviders.tsx:KYCComPilotProvider- uses KYC challenge functionSignatureGatingComPilotProvider- uses signature gating challenge functionUpdated
_app.tsx:Modified
/api/challengeendpoint:workflowTypefrom request bodyProblem Solved
Previously, the workflow ID was being sent in two different places with potentially different values. This refactoring ensures that:
WORKFLOW_KYC_IDWORKFLOW_GATING_IDTest plan
🤖 Generated with Claude Code