Skip to content

Conversation

@ekzyis
Copy link
Member

@ekzyis ekzyis commented Nov 2, 2025

TODO:

  • ✅ remove settings from flow
  • 🚧 find new location for button to delete wallet

Note

Removes the settings step from the wallet multi-step flow, adds first/last-step hooks, and saves the wallet (navigating to /wallets) on the final step with updated navigator buttons.

  • Wallet Flow:
    • Remove SETTINGS step; flow now only send and receive (wallets/client/components/form/index.js).
    • On last step, submit triggers useSaveWallet and navigates to /wallets; navigator shows “save” instead of “next” (index.js).
    • Navigator logic simplified: show cancel on first step, back otherwise; skip only when not last (index.js).
  • Hooks:
    • Add useIsFirstStep and useIsLastStep to components/multi-step-form.js and use them in form navigator.
    • Enhance useSaveWallet to push to /wallets after upsert (wallets/.../hooks.js).
  • Settings:
    • Stop redirecting from Settings submit handler (settings.js), as settings are no longer part of the flow.

Written by Cursor Bugbot for commit d5e250f. This will update automatically on new commits. Configure here.

@ekzyis ekzyis marked this pull request as draft November 2, 2025 10:58
@ekzyis ekzyis force-pushed the remove-wallet-settings-from-flow branch from d5e250f to a7f766b Compare November 2, 2025 12:58
@ekzyis ekzyis force-pushed the remove-wallet-settings-from-flow branch from a7f766b to e959790 Compare November 2, 2025 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants