Skip to content

Re-triggers the Tour on Every Mount for Returning Users - #956

Open
Abuh-Bakar wants to merge 4 commits into
FinChippay:mainfrom
Abuh-Bakar:new-branch
Open

Re-triggers the Tour on Every Mount for Returning Users#956
Abuh-Bakar wants to merge 4 commits into
FinChippay:mainfrom
Abuh-Bakar:new-branch

Conversation

@Abuh-Bakar

Copy link
Copy Markdown
Contributor

Implemented the onboarding dismissal persistence fix.

Updated:

  • onboardingState.ts: added dismissed and completedAt, persistence helpers, reset cleanup, and show checks.
  • useOnboardingTour.ts: hydrates canonical state and persists completion/dismissal.
  • OnboardingTour.tsx: active action now says “Skip and don't show again” and permanently dismisses.
  • onboarding.test.tsx: added persistence and remount coverage.

Made changes.

Closes #903

@github-actions github-actions Bot added the needs-review PR ready for Greptile AI code review label Aug 27, 2026
@github-actions

Copy link
Copy Markdown

🤖 Greptile AI Code Review

Greptile will automatically review this PR (4 file(s) changed).

Review gates:

  • ✅ CodeQL Security Scan
  • ✅ Custom rules (.greptile/config.json)
  • ✅ Architecture guidelines (.greptile/rules.md)

To manually trigger a re-review, comment @greptileai on this PR.
To skip review, add the skip-review label.

}

const EMPTY_PROGRESS: OnboardingProgress = {
version: ONBOARDING_STATE_VERSION,
completedSteps: [],
completed: false,
completedAt: null,
dismissed: false,
lastSeen: 0,
featureVersions: {},
};

@Abuh-Bakar

Copy link
Copy Markdown
Contributor Author

Kindly merge pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review PR ready for Greptile AI code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

#126 — Re-triggers the Tour on Every Mount for Returning Users (No Dismiss Persistence)

2 participants