Skip to content

Remove settings from flow#2628

Closed
ekzyis wants to merge 1 commit intomasterfrom
remove-wallet-settings-from-flow
Closed

Remove settings from flow#2628
ekzyis wants to merge 1 commit intomasterfrom
remove-wallet-settings-from-flow

Conversation

@ekzyis
Copy link
Copy Markdown
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 d5e250f928736c5e8420b0b0785be3191ef5ec3d. 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
@huumn huumn closed this Jan 11, 2026
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