Skip to content

Conversation

@Boy132
Copy link
Member

@Boy132 Boy132 commented Jan 9, 2026

Summary by CodeRabbit

  • Documentation
    • Expanded guidance on registering plugin routes with RouteServiceProvider, including practical examples for single endpoints, loading routes from external files, and client API routes.
    • Documentation content added in two locations to improve discoverability and access.

✏️ Tip: You can customize this high-level summary in your review settings.

@Boy132 Boy132 self-assigned this Jan 9, 2026
@netlify
Copy link

netlify bot commented Jan 9, 2026

Deploy Preview for pelica ready!

Name Link
🔨 Latest commit 3848c2f
🔍 Latest deploy log https://app.netlify.com/projects/pelica/deploys/6960c8b228eb5a0008539370
😎 Deploy Preview https://deploy-preview-188--pelica.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link

coderabbitai bot commented Jan 9, 2026

📝 Walkthrough

Walkthrough

Two duplicated "Routes" documentation blocks were added to docs/panel/advanced/plugins.mdx, showing how to register plugin routes via a RouteServiceProvider with examples for single endpoints, loading routes from a file, and adding client API routes.

Changes

Cohort / File(s) Summary
Documentation Updates
docs/panel/advanced/plugins.mdx
Inserted two identical "Routes" documentation blocks (+~28–82 lines total depending on diff view) demonstrating RouteServiceProvider usage: single endpoint registration, loading routes from a file, and API route examples with middleware/prefix.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 I hop through docs with pen in paw,
Two mirrored Routes, now plain to saw.
From file to endpoint, paths align,
Plugin ways laid out, neat and fine. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Add info about routes to plugin docs' accurately describes the main change—documentation additions about plugin routes in the docs/panel/advanced/plugins.mdx file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1256f86 and 3848c2f.

📒 Files selected for processing (1)
  • docs/panel/advanced/plugins.mdx
🚧 Files skipped from review as they are similar to previous changes (1)
  • docs/panel/advanced/plugins.mdx

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In @docs/panel/advanced/plugins.mdx:
- Line 223: The Route definition in the docs contains an extra closing
parenthesis causing a PHP syntax error; update the Route::get example that
references TestController::class and the 'test' method to remove the trailing
extra ')' so the call to ->name('my-plugin.test') ends with a single closing
parenthesis only, ensuring the Route::get(...)->name(...) statement is
syntactically valid.
- Around line 210-236: The Route example has a stray closing parenthesis in the
Route::get line: change Route::get('test', [TestController::class,
'test'])->name('my-plugin.test')); to remove the extra ')' so the line ends with
->name('my-plugin.test'); ensuring the array [TestController::class, 'test'] and
the method chain terminate correctly.
📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 804b8e4 and 1256f86.

📒 Files selected for processing (1)
  • docs/panel/advanced/plugins.mdx

@Boy132 Boy132 merged commit 348a8fd into main Jan 9, 2026
5 checks passed
@Boy132 Boy132 deleted the boy132/plugin-routes branch January 9, 2026 12:36
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.

3 participants