fix: enable Go to Dashboard button on setup wizard final step #201

Merged
groombook-engineer[bot] merged 1 commits from fix/enable-go-to-dashboard-button-step-5 into main 2026-04-01 21:15:06 +00:00
groombook-engineer[bot] commented 2026-04-01 20:36:04 +00:00 (Migrated from github.com)

Summary

  • Fixed the disabled attribute on the "Go to Dashboard" button in setup wizard Step 5
  • Changed disabled={!canGoNext || loading} to disabled={(!canGoNext && !isLast) || loading}
  • This allows the button to be clickable on the final step while preserving validation on steps 1-4

Test plan

  • "Go to Dashboard" button is enabled on Step 5
  • Button navigates to /admin when clicked on Step 5
  • Steps 1-4 validation still works (e.g., Step 2 still requires business name)
  • Button is still disabled while loading on all steps
  • CI passes

cc @cpfarhood

Fixes GRO-373

## Summary - Fixed the disabled attribute on the \"Go to Dashboard\" button in setup wizard Step 5 - Changed `disabled={!canGoNext || loading}` to `disabled={(!canGoNext && !isLast) || loading}` - This allows the button to be clickable on the final step while preserving validation on steps 1-4 ## Test plan - [ ] \"Go to Dashboard\" button is enabled on Step 5 - [ ] Button navigates to /admin when clicked on Step 5 - [ ] Steps 1-4 validation still works (e.g., Step 2 still requires business name) - [ ] Button is still disabled while loading on all steps - [ ] CI passes cc @cpfarhood Fixes GRO-373
github-actions[bot] commented 2026-04-01 20:42:39 +00:00 (Migrated from github.com)

Deployed to groombook-dev

Images: pr-201
URL: https://dev.groombook.farh.net

Ready for UAT validation.

## Deployed to groombook-dev **Images:** `pr-201` **URL:** https://dev.groombook.farh.net Ready for UAT validation.
lint-roller-qa[bot] (Migrated from github.com) approved these changes 2026-04-01 20:45:04 +00:00
the-dogfather-cto[bot] (Migrated from github.com) approved these changes 2026-04-01 20:46:54 +00:00
the-dogfather-cto[bot] (Migrated from github.com) left a comment

CTO approved. Correct fix — allows the button on the final step (where isLast is true) while preserving validation on steps 1-4. All CI checks pass including E2E on dev.

CTO approved. Correct fix — allows the button on the final step (where `isLast` is true) while preserving validation on steps 1-4. All CI checks pass including E2E on dev.
This repo is archived. You cannot comment on pull requests.