Skip to content

default Vault after Signup#13

Merged
knivram merged 7 commits into
mainfrom
Vault-after-signup
Jul 5, 2025
Merged

default Vault after Signup#13
knivram merged 7 commits into
mainfrom
Vault-after-signup

Conversation

@ErionKsv
Copy link
Copy Markdown
Collaborator

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
password-safe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 5, 2025 10:17pm

Comment thread app/actions/_userActions.ts Outdated
Comment thread app/actions/_userActions.ts Outdated
Comment thread app/actions/_userActions.ts Outdated
Comment thread app/actions/_vaultActions.ts Outdated
Comment thread app/actions/_vaultActions.ts Outdated
Comment thread app/actions/_userActions.ts Outdated
Comment thread app/actions/_userActions.ts Outdated
Comment thread app/actions/_userActions.ts Outdated
Comment thread app/actions/_userActions.ts Outdated
Comment thread app/actions/_userActions.ts Outdated
Comment thread app/actions/_userActions.ts Outdated
Comment thread components/auth/onboarding-form.tsx Outdated

try {
const { publicKey, wrappedPrivateKey, salt } =
const { publicKey, wrappedPrivateKey, salt, generateAndWrapVaultKey } =
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This variable needs to be renamed too...

Suggested change
const { publicKey, wrappedPrivateKey, salt, generateAndWrapVaultKey } =
const { publicKey, wrappedPrivateKey, salt, wrappedDefaultVaultKey } =

@knivram knivram merged commit 88d43d9 into main Jul 5, 2025
1 check passed
@knivram knivram deleted the Vault-after-signup branch July 5, 2025 22:16
@knivram knivram restored the Vault-after-signup branch July 8, 2025 09:29
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