Skip to content

Add import course#96

Open
andreia wants to merge 7 commits into4.xfrom
import_csv
Open

Add import course#96
andreia wants to merge 7 commits into4.xfrom
import_csv

Conversation

@andreia
Copy link
Member

@andreia andreia commented Feb 28, 2026

Details

Add import course.


Note

Medium Risk
Medium risk due to new background import path that creates multiple related records in a transaction and a schema-changing migration that alters an existing column type.

Overview
Adds an Import Course action to the courses list that uploads a CSV, stores it temporarily, and dispatches a queued ImportCourseFromCsv job to create a new course from the file.

Introduces CourseStepsImport (Laravel Excel) to parse supported CSV formats and create lessons/steps plus associated materials (video/image/link), including URL extraction/conversion for video embeds and fallback behavior when lesson names are missing.

Separately updates Video::provider detection to recognize more YouTube/Vimeo URL variants and return external otherwise, and adds a migration (change_name_to_text_on_lms_tests_table) to change lms_tests.name from string to text (with rollback). Tests and fixtures were added/updated to cover the new import and provider behaviors.

Written by Cursor Bugbot for commit 6c1f627. This will update automatically on new commits. Configure here.

@andreia andreia requested a review from swilla February 28, 2026 02:35
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Copy link
Member

@swilla swilla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good!

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.

2 participants