Skip to content

New plugin config option for all plugins#37

Merged
tungleduyxyz merged 3 commits into
masterfrom
kaui_5.27
Jun 3, 2026
Merged

New plugin config option for all plugins#37
tungleduyxyz merged 3 commits into
masterfrom
kaui_5.27

Conversation

@tungleduyxyz

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Copy link
Copy Markdown

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 support in the KPM UI to view and upload per-tenant plugin configuration for (running) plugins, aligning with the goal of moving plugin configuration into KPM.

Changes:

  • Adds a new POST route and controller action to upload tenant plugin configuration.
  • Fetches existing PLUGIN_CONFIG_ tenant keys and passes them to the plugins UI.
  • Adds a “Config” action in the plugins list and renders a Bootstrap modal with an upload form, plus minimal styling.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
config/routes.rb Adds the POST endpoint to upload plugin configuration.
app/controllers/kpm/nodes_info_controller.rb Fetches existing tenant plugin config on index; implements the upload action.
app/views/kpm/nodes_info/index.html.erb Passes tenant plugin config data into the plugins partial.
app/views/kpm/nodes_info/_official_plugins.html.erb Adds “Config” link and renders config modals.
app/views/kpm/nodes_info/_form_plugin_config.erb New modal + form for editing/uploading plugin config.
app/assets/stylesheets/kpm/nodes_info.css Adds styling for the plugin config modal elements.

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

Comment thread app/views/kpm/nodes_info/_form_plugin_config.html.erb
Comment thread app/views/kpm/nodes_info/_official_plugins.html.erb Outdated
Comment thread app/controllers/kpm/nodes_info_controller.rb Outdated
Comment thread app/controllers/kpm/nodes_info_controller.rb
Comment thread app/controllers/kpm/nodes_info_controller.rb
@tungleduyxyz tungleduyxyz merged commit 037ef9e into master Jun 3, 2026
9 checks passed
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