fix(#629): surface fee-payer derivation mode and warn on PRF fallback - #664
Open
Hussman256 wants to merge 3 commits into
Open
fix(#629): surface fee-payer derivation mode and warn on PRF fallback#664Hussman256 wants to merge 3 commits into
Hussman256 wants to merge 3 commits into
Conversation
…F fallback - Add 'Fee Payer' card to Settings overview linking to /settings/fee-payer - Shows a red dot indicator and red border when isFeePayerPrfDowngrade() is true - Subtitle changes to 'PRF unavailable - using legacy derivation' on downgrade - Add PRF-downgrade banner on Dashboard (dismissible) - Appears after ensureFeePayer() settles when PRF was requested but fell back - Links to /settings/fee-payer where the full derivation log and copy-for-bug-report button live - /settings/fee-payer page (already existed) now reachable without a debugger: - Shows active G... address and derivation mode (prf-raw / prf-hkdf / legacy) - Shows every candidate that was probed and its on-chain probe result - Downgrade warning with error detail if prfError is present - One-click copy of the full diagnostics blob for bug reports
- Hero: centered biometric pulse ring (two-ring gold animation) wrapping the VeilMark at 96px, defined as .biometric-pulse in globals.css - Headline: Lora-italic h1 with 'your wallet.' highlighted via .hl gold underlay - Lede: single sentence explaining no seed phrase, biometric, Stellar - CTAs: Create wallet (btn-gold, id=onboarding-create) and Recover existing wallet (btn-ghost, id=onboarding-recover) - Footnote: 'Secured by WebAuthn on Stellar' id=webauthn-footnote (was mis-IDed as tutorial-title) - Removed dead empty-card comment block - Onboarding tutorial + registering/deploying/done flow unchanged
…epayer-derivation-visibility-629
|
@Hussman256 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
|
@Hussman256 is attempting to deploy a commit to the miracle656's projects Team on Vercel. A member of the Team first needs to authorize it. |
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
1 participant
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
Closes #629. Closes #617.
fix(#629) — Surface fee-payer derivation and warn on PRF fallback
The
/settings/fee-payerdetail page (active G… address, derivation mode, probe log, copy-for-bug-report) was already fully implemented but completely unreachable — nothing in the UI linked to it, and nothing warned when PRF silently downgraded to legacy.Changes
Settings overview — Fee Payer card
/settings/fee-payerisFeePayerPrfDowngrade()is true: red dot indicator, red-tinted border, subtitle changes to "PRF unavailable — using legacy derivation"Dashboard — PRF downgrade banner
ensureFeePayer()settles, surfaces a dismissible red banner when PRF fell back to legacy/settings/fee-payer/settings/fee-payer(pre-existing, now reachable)prf-raw/prf-hkdf/legacy)fix(#617) — Redesign the onboarding / create screen
Restyles
app/page.tsxto match the redesign spec. All logic (passkey register → fee-payer → factory deploy) is unchanged.Changes
app/globals.css@keyframes pulse-ring+@keyframes pulse-ring-2— two staggered gold rings expanding outward from the mark.biometric-pulseclass with::before/::afterpseudo-elements for the ringsapp/page.tsx.biometric-pulsecontainer wrapping<VeilMark size={64} /><h1>— "Your passkey is your wallet."* — with"your wallet."in the.hlgold highlightid="onboarding-create"(Create wallet) andid="onboarding-recover"(Recover existing wallet)tutorial-title→webauthn-footnote){/* Main card */}comment blockAcceptance criteria
#629
✅ Fee Payer card in Settings visible on every session
✅ PRF-downgraded users see a warning on Dashboard and the Settings card
✅ Detail page shows G… address, derivation mode, and probe log with one-click copy
#617
✅ Centered hero with pulse ring + gold-highlighted Lora headline
✅ Create / Recover CTAs; WebAuthn footnote
✅ Onboarding tutorial + create flow unchanged
✅ Dead markup removed