feat: course-linked discussions — per-lesson comment panels#792
Closed
clbotdev wants to merge 2 commits into
Closed
feat: course-linked discussions — per-lesson comment panels#792clbotdev wants to merge 2 commits into
clbotdev wants to merge 2 commits into
Conversation
…payment config Fixes #583 - Added PAYMENT_METHOD_NONE as the first option in the payment method dropdown - Removed placeholderMessage since 'None' is now a selectable option - Users can now reset their payment configuration by selecting 'None'
Enable per-lesson discussions by linking Communities to Courses. Data Model: - Add discussions + discussionCommunityId fields to Course - Add courseId field to Community (for course-linked communities) - Add lessonId field to CommunityPost (for lesson-linked posts) - Add COURSE_DISCUSSION_* notification entity actions Backend: - Auto-create Community when discussions toggled ON in course settings - Auto-create CommunityPost per lesson (syncs title, soft-deletes on removal) - Cascade soft-delete discussion community on course deletion - Filter course-linked communities from main community listing - New GraphQL queries: getCourseDiscussionPosts, getCourseDiscussionStream - Notification routing: course discussion events get deep-linked to lesson URL - Auto-enroll students into discussion community on course enrollment Frontend: - Course Manage page: Discussions toggle component (Switch) - Lesson page: Desktop sidebar (340px) + mobile slide-in FAB/drawer - LessonDiscussionPanel: Comments, threaded replies, likes via existing APIs - use-product hook updated to fetch discussions + discussionCommunityId
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
2 participants
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
Enables per-lesson discussions by linking Communities to Courses. Students can comment on each lesson directly from the lesson player — no more switching between lesson content and a separate discussion board.
How it works
Changes (22 files, +1,329 lines)
Data Model:
Backend:
Frontend:
Design
See PRD: docs/prd-course-discussions.md