Skip to content

Implement course learning and wallet onboarding flows - #501

Merged
DeFiVC merged 1 commit into
ChainLearnOfficial:mainfrom
Queenode:fix/issues-349-352-learning-flows
Sep 28, 2026
Merged

DeFiVC merged 1 commit into
ChainLearnOfficial:mainfrom
Queenode:fix/issues-349-352-learning-flows

Conversation

@Queenode

Copy link
Copy Markdown
Contributor

Summary

  • Build the module viewer for text, video, and structured interactive content, with progress display, module-specific completion tracking, previous/next navigation, and the quiz action after completion.
  • Implement the four-step onboarding sequence for display name, learning goals, pace and language preferences, and profile review. Validate before advancing, persist preferences through the profile API, and provide a dashboard skip action.
  • Guide wallet connection through Freighter detection, wallet access, challenge signing, and verification, with profile-aware redirects and actionable classified errors.
  • Harden text truncation for short limits and add regression coverage for content parsing and truncation edge cases.

Validation

  • ESLint passes for all changed files.
  • Focused Vitest suites pass: 37 tests.
  • git diff --check passes.
  • Full typecheck is blocked by existing syntax errors in src/lib/api/client.test.ts and src/lib/hooks/use-wallet-watcher.test.tsx.

Closes #349
Closes #350
Closes #351
Closes #352

@netlify

netlify Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for chainlearn failed.

Name Link
🔨 Latest commit 0cf6e89
🔍 Latest deploy log https://app.netlify.com/projects/chainlearn/deploys/6aba4e75dd90e600081f2a5f

@drips-wave

drips-wave Bot commented Sep 28, 2026

Copy link
Copy Markdown

@Queenode 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

@DeFiVC
DeFiVC merged commit e388e14 into ChainLearnOfficial:main Sep 28, 2026
0 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants