Skip to content

Add support to hook configuration#2

Open
silasrm wants to merge 3 commits intosolutionforest:mainfrom
silasrm:silasrm-patch-1
Open

Add support to hook configuration#2
silasrm wants to merge 3 commits intosolutionforest:mainfrom
silasrm:silasrm-patch-1

Conversation

@silasrm
Copy link
Copy Markdown

@silasrm silasrm commented Dec 12, 2025

No description provided.

@lam0819 lam0819 requested a review from Copilot February 2, 2026 21:58
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds configuration support for customizing the render hook placement of the header select component. Previously, the hook was hardcoded to TOPBAR_START; now it can be configured via the config file while maintaining the same default behavior.

Changes:

  • Added configurable hook placement via default_hook config option
  • Removed hardcoded agent_config option provider registration
  • Added PanelsRenderHook import to config file

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/HeaderSelectPlugin.php Updated to use configurable hook instead of hardcoded value; removed agent_config provider
config/filament-header-select.php Added default_hook configuration option with default value

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants