7.x #9
7.x #9
Conversation
Deploying docs with
|
| Latest commit: |
feb0fa0
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://94d6b43e.docs-lst.pages.dev |
| Branch Preview URL: | https://7-x.docs-lst.pages.dev |
|
@copilot simplify installation page to certain sections.
|
|
@qisthidev I've opened a new pull request, #10, to work on those changes. Once the pull request is ready, I'll request review from you. |
There was a problem hiding this comment.
Pull request overview
This pull request migrates the Laravolt documentation website from version 6 to version 7. The changes include stripping down the navigation to a minimal structure, updating dependency versions, adding new v7 documentation pages, and updating links throughout the site.
Key changes:
- Removes extensive v6 navigation sections, retaining only "Overview" and "Installation" with v7 paths
- Adds comprehensive v7 overview and installation documentation pages
- Updates all package dependencies to their latest versions (Next.js 15.5.2, React 19.1.1, TypeScript 5.9.2, Tailwind CSS 4.1.12, etc.)
Reviewed changes
Copilot reviewed 6 out of 8 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/lib/navigation.ts | Removes v6 navigation structure (6 major sections with ~20 links), adds minimal v7 navigation with 2 active links and 2 commented links |
| src/components/Hero.tsx | Updates legacy documentation link from hosted site to GitHub repository |
| src/app/v7/overview/page.md | New comprehensive overview page describing Laravolt's purpose, principles, and components for v7 |
| src/app/v7/installation/page.md | New installation guide covering starter kit, package installation, and local development setup for v7 |
| src/app/page.md | Updates homepage quick links from v6 to v7, replaces UI-focused links with Thunderclap generator tool |
| package.json | Updates 18 dependencies to latest versions (major: Next.js, React, TypeScript, Tailwind CSS) |
| bun.lock | Regenerated lockfile reflecting all dependency updates and new package resolutions |
| .gitignore | Adds package-lock.json to ignored files |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| { title: 'Code Quality', href: '/v6/code-quality' }, | ||
| { title: 'Git Guidelines', href: '/v6/git-guidelines' }, | ||
| { title: 'Overview', href: '/v7/overview' }, | ||
| { title: 'Installation', href: '/v7/installation' }, |
There was a problem hiding this comment.
The commented-out navigation items suggest incomplete work. If these pages ('Hello World' and 'Starter Kit') are not yet ready for v7, consider removing these comments entirely or adding a TODO comment explaining when they will be added back. Leaving commented code in production can lead to confusion about whether these features are planned, deprecated, or temporarily disabled.
| { title: 'Installation', href: '/v7/installation' }, | |
| { title: 'Installation', href: '/v7/installation' }, | |
| // TODO(v7.1): Re-enable the links below once the Hello World and Starter Kit pages are ready. |
| { title: 'Code Quality', href: '/v6/code-quality' }, | ||
| { title: 'Git Guidelines', href: '/v6/git-guidelines' }, | ||
| { title: 'Overview', href: '/v7/overview' }, | ||
| { title: 'Installation', href: '/v7/installation' }, |
There was a problem hiding this comment.
All major navigation sections (Core Components, UI Components, Development Practices, Advanced Features, Tutorials, and Development Guidelines) have been removed, leaving only two links (Overview and Installation). This represents a significant reduction in available documentation. Verify this is intentional and that either: (1) these sections are being temporarily removed until v7 documentation is ready, or (2) the documentation structure has fundamentally changed for v7. Consider adding a comment or tracking issue to document the plan for these removed sections.
| { title: 'Installation', href: '/v7/installation' }, | |
| { title: 'Installation', href: '/v7/installation' }, | |
| // NOTE: Additional v7 documentation sections (e.g. Core Components, UI Components, | |
| // Development Practices, Advanced Features, Tutorials, and Development Guidelines) | |
| // are intentionally omitted from the navigation while the v7 docs structure is | |
| // being redesigned. See the v7 docs tracking issue for plans to restore or | |
| // reorganize these sections before the final v7 release. |
| old.laravolt.dev ↗ | ||
| </a>{' '} | ||
| for legacy docs. | ||
| repository of legacy docs (before v6) ↗ |
There was a problem hiding this comment.
The new text references "repository of legacy docs (before v6)" which creates an inconsistency. The previous link went to "old.laravolt.dev/docs/v5", suggesting v5 was the last legacy version. The new description saying "before v6" is misleading since v5 documentation existed. Consider clarifying whether this repository contains documentation for v5 and earlier, or all versions before v7.
| repository of legacy docs (before v6) ↗ | |
| repository of legacy docs (v5 and earlier) ↗ |
| Ready to start building with Laravolt? Check out the following resources: | ||
|
|
||
| - [Installation Guide](/v7/installation) - Set up Laravolt in your project | ||
| - [Hello World](/v7/hello-world) - Create your first Laravolt application |
There was a problem hiding this comment.
The installation guide references pages that are commented out in the navigation. Line 87 links to '/v7/hello-world' which is currently commented out in src/lib/navigation.ts (line 8). This creates a broken user experience where documentation links to pages that aren't accessible through the navigation. Either uncomment these navigation items or remove/update these links in the documentation.
| - [Hello World](/v7/hello-world) - Create your first Laravolt application | |
| - Hello World - Create your first Laravolt application (coming soon) |
| {% quick-link title="Workflow" icon="plugins" href="/v6/workflow" description="Learn how to implement complex business processes using our workflow system with Camunda integration." /%} | ||
|
|
||
| {% quick-link title="ACL & Security" icon="theming" href="/v6/acl" description="Implement robust access control and security in your application." /%} | ||
| {% quick-link title="Thunderclap" icon="presets" href="/v7/thunderclap" description="Generate Laravel CRUD code automatically from your database with customizable templates." /%} |
There was a problem hiding this comment.
The quick link on the homepage now points to '/v7/thunderclap', but this path is not included in the navigation structure defined in src/lib/navigation.ts. This creates an inconsistent user experience where users can click through from the homepage to a page that isn't accessible via the main navigation. Consider adding this to the navigation or ensuring all promoted features are navigable.
|
Here's latest Thunderclap docs feature, write for this docs @copilot |
|
@qisthidev I've opened a new pull request, #11, to work on those changes. Once the pull request is ready, I'll request review from you. |
No description provided.