Skip to content

Integrate vetkeys to current app#2

Open
Huygon764 wants to merge 1 commit into
Quantum3-Labs:developfrom
Huygon764:integrate-vetkeys
Open

Integrate vetkeys to current app#2
Huygon764 wants to merge 1 commit into
Quantum3-Labs:developfrom
Huygon764:integrate-vetkeys

Conversation

@Huygon764
Copy link
Copy Markdown

@Huygon764 Huygon764 commented Sep 15, 2025

The idea is:

  1. Switch from sync to async, since we need to call the vetkey API (which is async).
  2. Use wallet_id instead caller() for data encryption, because I want a shared key to decrypt all data of 1 wallet. Otherwise, we would need n separate signer encryptions, which would waste memory.

If you want to deploy it to mainnet, remember to add "init_arg = production" from dfx.json:
"q3x_backend": {
"candid": "src/q3x_backend/q3x_backend.did",
"package": "q3x_backend",
"type": "rust",
"init_arg": ""production""
},

Check full flow demo below:
https://www.loom.com/share/88f0ee7b604a4578b7253e487efc78d6?sid=a72eff55-4642-45ad-bc67-9e42ff75ecb7

Copy link
Copy Markdown
Collaborator

@gianalarcon gianalarcon left a comment

Choose a reason for hiding this comment

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

Initial review, Need to fix few comments

Comment thread src/q3x_backend/src/lib.rs
Comment thread src/q3x_backend/src/lib.rs
Comment thread src/q3x_backend/src/lib.rs
Comment thread src/q3x_backend/src/lib.rs
Comment thread src/q3x_backend/src/lib.rs
Comment thread src/q3x_backend/src/lib.rs
Comment thread src/q3x_backend/src/lib.rs
Comment thread src/q3x_backend/src/lib.rs
Comment thread src/q3x_backend/src/lib.rs
Comment thread src/q3x_backend/src/lib.rs
@Huygon764 Huygon764 changed the base branch from main to develop September 16, 2025 16:08
@Huygon764 Huygon764 force-pushed the integrate-vetkeys branch 2 times, most recently from 94405dc to 05894b6 Compare September 17, 2025 08:12
Copy link
Copy Markdown
Collaborator

@gianalarcon gianalarcon left a comment

Choose a reason for hiding this comment

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

We commented tests for now, but we can enable it if we really need them to speed-up integration

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