Change unique course fields per tenant

[andreia]

Details

Change unique course fields (name, slug, and external_id) per tenant.

php artisan vendor:publish --tag=filament-lms-migrations
php artisan migrate

---

Note

Medium Risk
Schema migration changes uniqueness rules on a core table; misconfiguration or partial deploy could affect course creation, but behavior is gated on tenancy config and covered by tests.

Overview
Enables separate tenants to reuse the same course name, slug, and external_id without colliding, aligning DB constraints and admin validation with multi-tenant LMS usage.

When tenancy is enabled and the tenant column exists on lms_courses, a new published migration drops global unique indexes on those three columns and adds composite uniques on [tenant_column, column]. The migration is idempotent on re-run, skips work when tenancy is off, and intentionally leaves down() empty so rollback does not try to restore global uniques after cross-tenant duplicates may exist.

Course admin forms now use scopedUnique instead of unique on name, external_id, and slug so Filament validation matches the tenant-scoped indexes. Feature tests cover index shape, no-op when tenancy is disabled, and rollback behavior.

^{Reviewed by Cursor Bugbot for commit febf572. Bugbot is set up for automated code reviews on this repo. Configure here.}

[cursor]

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

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

^{Reviewed by Cursor Bugbot for commit 1ac487e. Configure here.}