Open
Conversation
There was a problem hiding this comment.
Pull request overview
This PR implements mobile bridge functionality to enable native mobile app integration with the web-based onboarding flow. It adds TypeScript type definitions and a wrapper library for communicating with native mobile features via JavaScript bridge, updates the onboarding components to support both web and mobile modes, creates a dedicated mobile onboarding route, and adds a Literature course mapping with associated icon assets.
Changes:
- Implements mobile bridge API for native app integration (OAuth handling, external URL opening, onboarding completion callbacks)
- Adds mode-based onboarding support to differentiate web vs mobile behavior
- Adds Literature course metadata with keyword matching and course icon
Reviewed changes
Copilot reviewed 7 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/types/mobileBridge.d.ts | Global type declarations for window.mobileBridge API |
| src/lib/mobileBridge.ts | Mobile bridge wrapper library with error handling |
| src/components/onboarding/OnboardingController.tsx | Added mobile mode support with session expiry UI and bridge integration |
| src/components/onboarding/ConnectLmsStep.tsx | Added mobile OAuth flow via bridge API |
| src/components/LayoutWrapper.tsx | Added bypass for mobile onboarding route |
| src/app/onboarding/page.tsx | Explicitly set web mode |
| src/app/mobile/onboarding/page.tsx | New mobile onboarding page with mobile mode |
| public/meta/course-match.json | Added Literature course mapping |
| public/icons/courses/text.book.closed.svg | New book icon for Literature course |
| public/icons/courses/pencil.line.svg | New pencil icon (unused) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Testing