Fix technical inaccuracies and update documentation for release#163
Open
lcharette wants to merge 4 commits into
Open
Fix technical inaccuracies and update documentation for release#163lcharette wants to merge 4 commits into
lcharette wants to merge 4 commits into
Conversation
- Fix duplicate 'use use' keyword typo in user-accounts code example - Replace outdated UF4/UF5 TokenRepository path with correct UF6 namespace reference - Update Laravel 8.x doc links to 11.x (model events, soft deletes) - Replace non-existent UserPageAction class reference with correct UsersSprunjeAction - Add missing semicolons to PHP property declarations in access-control examples - Add missing never() callback to access conditions table - Fix broken anchor #Accessconditions -> #access-conditions in groups page - Fix critical lastActivity() API misuse: method returns ?Activity, not a Builder - Fix logging example missing required user_id context key (causes LogicException) - Fix incorrect note claiming lastActivity() with parens returns a relationship - Fix DI override example: use UserActivityLoggerInterface::class key, correct constructor Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Fixes identified during full pre-release documentation review: - Installation: remove beta stability flag and TODO comments, fix npm script (dev → vite:dev), fix Docker Compose V2 syntax (docker-compose → docker compose), rename docker-compose.yml → compose.yaml, fix MySQL port typo (8593 → 8503), fix composer update → install - Structure: Symfony Console 5 → 6 with updated link; three sprinkles (not four); fix i81n → I18n typo - Sprinkles: fix /app/tests path, add MarkdownExtensionRecipe to optional recipes table, fix vendor path - DI: fix possessive its/it's, update stale Laravel docs links (8.x → versionless), fix getServices() description - Routes: fix Slim 3 array $args pattern, fix return $response anti-pattern, fix @return docblock, remove TODO comment, fix validate() return type description - Pages: fix {% urlFor %} → {{ urlFor }} (Twig function vs tag), align template path prose/code - Config: fix nested array merge example (timezone stays under php key) - i18n: fix locale paths (UF4/5 → UF6 format), fix es_ES typo, update monorepo links - Mail: fix send() bool parameter semantics - UI Theming: fix $validate() return type handling; fix AlertInterface import path - Advanced: fix PHP namespace leading backslash, implements → extends for ExceptionHandler, fix ResourceStream syntax errors, replace broken middleware image with link, add obsolete: true to webpack-encore sub-pages, fix alert-stream deprecation claim - Troubleshooting: update Postman link, remove commented-out UF4 content Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ix image path - Sprinkles: restore Pink Cupcake section; update count to '3 sprinkles and one theme package' - DI: pin Laravel docs links to 10.x (cache, session) - Users: pin Laravel docs links to 10.x (eloquent events, soft deletes) - i18n: clarify override example references the Core sprinkle file - Alert stream: restore PHP-side deprecation warning - Middlewares: restore image, move to images/ subfolder, fix path to /images/middleware.png Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #163 +/- ##
===========================================
Coverage 100.00% 100.00%
Complexity 207 207
===========================================
Files 28 28
Lines 889 889
Branches 10 13 +3
===========================================
Hits 889 889 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Fixes #160 Fixes #161 Fixes #162 Chapter 11 - Database (#160): - Migration namespace: V400 → v400 (lowercase, case-sensitive on Linux) - users.id column type: string → int (auto-increment) - System tables path: remove UF4/5 app/system/ path, reference Core sprinkle instead - Sprunje: add note explaining ->like() / ->orLike() are UF-specific builder methods Chapter 14 - CLI (#161): - assets:build: remove stale aliases (webpack, build-assets); rewrite description for Vite-first behavior - assets:webpack: fix npm script names (dev/build/watch → webpack:dev/webpack:build/webpack:watch); add --server option - assets:install / assets:update: packages.lock → package-lock.json - bake: remove create:admin-user from built-in list (added by Account sprinkle event listener) Chapter 15 - Assets/Vite (#162): - vite:build script: vite build → vue-tsc && vite build (TypeScript type-check runs before build) - coverage script: vitest --coverage → vitest run --coverage (prevents hanging watch mode in CI) - Add prose explaining vue-tsc type-check step Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This was
linked to
issues
May 27, 2026
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
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.
Address technical accuracy issues identified in the documentation, including correcting code examples, updating links to the latest Laravel version, and fixing typos. Ensure clarity and correctness across various chapters in preparation for the release.