Skip to content

KAUI 4.8 release#593

Open
tungleduyxyz wants to merge 7 commits intomasterfrom
kaui_4.08
Open

KAUI 4.8 release#593
tungleduyxyz wants to merge 7 commits intomasterfrom
kaui_4.08

Conversation

@tungleduyxyz
Copy link
Copy Markdown
Contributor

@tungleduyxyz tungleduyxyz commented Apr 5, 2026

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

Removes the single-node assumption when retrieving plugin information so multi-node Kill Bill deployments can display plugin data correctly, and adjusts account search flows to better handle multiple matches.

Changes:

  • Aggregate plugins across all nodes instead of using nodes_info.first (helper + test).
  • Deduplicate running plugin names across nodes when building the configurable plugin list.
  • Update account search/controllers to route users to the accounts list when a query yields multiple matches.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
test/functional/kaui/admin_tenants_controller_test.rb Updates test helper to aggregate plugins across all nodes.
app/helpers/kaui/plugin_helper.rb Aggregates plugin info across nodes and dedupes running plugins by name.
app/controllers/kaui/home_controller.rb Changes account search behavior for multiple matches (but introduces correctness/perf concerns).
app/controllers/kaui/accounts_controller.rb Changes index redirect logic for search results (but uses incorrect pagination metadata).

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

@tungleduyxyz tungleduyxyz changed the title Refactor plugin info retrieval to support multiple nodes display KAUI 4.8 release Apr 7, 2026
@tungleduyxyz tungleduyxyz force-pushed the kaui_4.08 branch 3 times, most recently from 223dbb3 to b1bd243 Compare April 7, 2026 15:56
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