Skip to content

Feature/onboarding checklist - #928

Open
Ademiitura wants to merge 4 commits into
ritik4ever:mainfrom
Ademiitura:feature/onboarding-checklist
Open

Feature/onboarding checklist#928
Ademiitura wants to merge 4 commits into
ritik4ever:mainfrom
Ademiitura:feature/onboarding-checklist

Conversation

@Ademiitura

Copy link
Copy Markdown
Contributor

Commit Type

  • feat: A new feature
  • fix: A bug fix
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code (formatting, etc.)
  • refactor: A code change that neither fixes a bug nor adds a feature
  • perf: A code change that improves performance
  • test: Adding missing or correcting existing tests
  • build: Changes affecting the build system or external dependencies
  • ci: Changes to CI/CD configuration files and scripts
  • chore: Other changes that don't modify src or test files
  • revert: Reverts a previous commit

Breaking change? (If yes, add BREAKING CHANGE in the commit footer)

  • Yes — this contains a breaking change
  • No

What changed

  • Adds an OnboardingChecklist component shown on first visit, listing 4 steps (connect wallet, create stream, monitor progress, claim tokens). Each step auto-checks based on real app state (wallet connection status, whether the user has created a stream, dashboard visit, and claim events) rather than being manually toggled.
  • Checklist is dismissible and re-openable via a new "? Help" button added to the header.
  • Dismissed/seen state persists in localStorage across sessions.
  • Also fixes several pre-existing issues blocking a clean local build/CI on this branch (same root causes found while working on [FEATURE] Add Storybook for component documentation #759): restored an empty frontend/package.json and package-lock.json, added the missing react-router-dom dependency (imported in App.tsx but never declared), fixed a syntax error in SenderDashboard.tsx, and fixed stale mock data types in StreamsTable.stories.tsx and WalletButton.stories.tsx.

Testing done

  • Ran the app locally (npm run dev) and manually verified the checklist renders on first visit, each step's checkbox reflects real state, clicking a step navigates to the relevant page, dismissing hides it and persists across refresh, and the "? Help" button reopens it.
  • Verified npx tsc --noEmit passes cleanly.

Related issues

Closes #751

Checklist

  • I used the correct commit type (see above) for the squashed commit message.
  • I kept the change focused on the related issue.
  • I added or updated tests where useful.
  • I updated documentation where behavior changed.
  • I verified the app still builds or explained why verification was skipped.
  • My commits follow Conventional Commits (enforced by commitlint).

- New OnboardingChecklist component: shows 4 steps (connect wallet,
  create stream, monitor progress, claim tokens), auto-checked based
  on real wallet/stream/event state via useOnboardingStatus hook
- Dismissible and re-openable via new Help button in header
- Dismissed and seen state persisted in localStorage

Also includes fixes needed to get a clean local build on this branch
(same root-cause issues found and fixed on feature/storybook-docs):
- Restored frontend/package.json and package-lock.json (empty on main)
- Added missing react-router-dom dependency (App.tsx imports it but
  it was never in package.json)
- Fixed SenderDashboard.tsx syntax error (missing if condition)
- Fixed StreamsTable.stories.tsx / WalletButton.stories.tsx type
  mismatches with current Stream/FreighterState shapes
@vercel

vercel Bot commented Aug 30, 2026

Copy link
Copy Markdown

@Ademiitura is attempting to deploy a commit to the ritik4ever's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Aug 30, 2026

Copy link
Copy Markdown

@Ademiitura 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! 🚀

Learn more about application limits

@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: aacf964a-dac2-4c8c-8865-07910abc49df


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

- Remove unused Legend import and eventsLoading state
- Remove console.log statements from story mock handlers
@Ademiitura

Copy link
Copy Markdown
Contributor Author

CI investigation notes:

Fixed as part of this PR (needed for a clean local build/lint/type-check on this branch, same root causes found while working on #759):

  • Restored an empty frontend/package.json and package-lock.json
  • Added the missing react-router-dom dependency (imported in App.tsx but never declared in package.json)
  • Fixed a syntax error in SenderDashboard.tsx (missing if condition)
  • Fixed stale mock data types in StreamsTable.stories.tsx and WalletButton.stories.tsx
  • Removed unused imports/state and console.log statements flagged by npm run lint in SenderDashboard.tsx and StreamsTable.stories.tsx

Pre-existing issues on main, out of scope for this PR — flagging rather than fixing blind:

  • npm run lint currently fails on main due to issues in files this PR doesn't touch:
    • CliffMarker.tsx — conditional useMemo call (Rules of Hooks violation)
    • FilterBar.stories.tsxuseState called inside a render function, not a component (Rules of Hooks violation)
    • CreateStreamForm.stories.tsx, StreamsTable.stories.tsx (unrelated lines) — unexpected console statements
    • A few react-hooks/exhaustive-deps warnings in CreateStreamForm.tsx, EditStartTimeModal.tsx, StreamMetricsChart.tsx, `StreamsTable.tsx

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Add onboarding checklist for new users

1 participant