fix(admin): one clear way to create a course for school admins (#665) - #666
Open
guillermoscript wants to merge 2 commits into
Open
fix(admin): one clear way to create a course for school admins (#665)#666guillermoscript wants to merge 2 commits into
guillermoscript wants to merge 2 commits into
Conversation
…bar (#665) A school admin left Sentry feedback "je ne vois pas comment faire cree des cours" (LMS-FRONT-9N) after wandering the settings pages for over a minute with no errors. The admin Courses page was a read-only stats + table view with no Create button, and the only creation entry point (/dashboard/teacher/courses/new) sat behind a "My Courses" sub-item that nothing labels as the place to create. - "Create Course" button in the admin Courses page header - Real empty state in the admin courses table with a create CTA when the school has no courses at all (filtered-to-empty keeps the plain message) - "Create Course" sub-item under Sidebar > Courses for admins - en/es copy; regression assertion in admin-pages.spec.ts Fixes LMS-FRONT-9N Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D6TeYjJcqunFTZ5FR52SDA
…mes wizard-only (#665) Admins had two creation paths that led to different places: the teacher course form (lands in the course editor) and the "Create Product" quick create (creates course + price, then dumps you on the Products list with no next step). The getting-started checklist pointed at the second one. - New /dashboard/admin/courses/new: quick create (title, free/paid, publish) + AI starter generator, breadcrumbed under Courses - Quick create now lands in the course editor so adding lessons is the obvious next step; "More options" opens the full product wizard - /dashboard/admin/products/new is the product wizard only (the ?advanced=1 split is gone; old links still land on the wizard) - Sidebar, Courses page, empty state, getting-started checklist and the onboarding wizard all point at the one page - Specs follow the routes; admin-crud's wizard-at-/products/new expectations are true again Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D6TeYjJcqunFTZ5FR52SDA
guillermoscript
changed the title
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
1 participant
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
Fixes #665 · Sentry user feedback LMS-FRONT-9N
A school admin on prod wrote "je ne vois pas comment faire cree des cours" after 1m18s of wandering between settings pages, no errors and no rage clicks.
Before: the admin Courses page had no Create button, and admins had two creation paths that behaved differently:
/dashboard/teacher/courses/new) → lands in the course editor./dashboard/admin/products/new) → creates course + price, then drops you on the Products list with no next step. The getting-started checklist pointed here.After: one page,
/dashboard/admin/courses/new, reachable from every admin surface, and it ends in the course editor where lessons get added.Changes
/dashboard/admin/courses/new: quick create (title, free/paid, price, publish or draft) + AI starter generator. Breadcrumb Admin → Courses → New Course./dashboard/teacher/courses/<id>), not the products list. "More options" opens the full product wizard./dashboard/admin/products/newis the product wizard only. The?advanced=1split is removed; old?advanced=1links still land on the wizard.admin-pages(checklist href + new CTA assertion),admin-product-course-creation(paths + quick-create now asserts the editor URL, then checks the product exists),plan-limit-surfaces(AI cap test on the new page).admin-crud's expectation that/products/newshows the wizard is true again.Uses base-ui
render={<Link/>}on Buttons, not a wrapping<Link>.QA
npm run typecheck✅ ·eslinton changed files ✅--workers=1:owner@e2etest.comondefault.lvh.me:3005:/dashboard/admin/courses/new./dashboard/admin/products/new→ multi-step wizard, no quick form.🤖 Generated with Claude Code
https://claude.ai/code/session_01D6TeYjJcqunFTZ5FR52SDA