Skip to content

[configuration] Show nested configuration settings - #10606

Open
Montekkundan wants to merge 6 commits into
aces:mainfrom
Montekkundan:montek/10471-config-tree-ui-clean
Open

[configuration] Show nested configuration settings#10606
Montekkundan wants to merge 6 commits into
aces:mainfrom
Montekkundan:montek/10471-config-tree-ui-clean

Conversation

@Montekkundan

Copy link
Copy Markdown
Contributor

Brief summary of changes

  • Builds configuration settings as a recursive tree under each configuration category.
  • Replaces the unfinished nested-setting placeholder with collapsible nested UI blocks.
  • Keeps existing leaf setting controls and submit behavior unchanged.

Note: This PR implements the tree/nested UI part of #10449 for #10471. Mapping field support is handled separately in #10458, and category separation is handled in #10457.

This PR is stacked on top of the config categories work from #10457. The #10471 specific change is the nested/tree UI rendering on top of that base.

Testing instructions

  • Open http://localhost:8080/configuration/ as a user with config permission.
  • Confirm the existing category tabs and settings still render.
  • Add or use a nested ConfigSettings.Parent test setting and confirm the parent group appears as a collapsible block.
  • Expand the group and confirm child settings render with the existing input controls.
  • If testing with a text child setting, confirm it renders as a text input. If testing with a boolean child setting, confirm it renders with the existing Yes/No controls.

Link(s) to related issue(s)

@Montekkundan Montekkundan self-assigned this Jun 16, 2026
@github-actions github-actions Bot added Language: SQL PR or issue that update SQL code RaisinBread PR or issue introducing/requiring improvements to the Raidinbread dataset Language: PHP PR or issue that update PHP code Module: configuration PR or issue related to configuration module labels Jun 16, 2026
@skarya22 skarya22 added the State: Needs work PR awaiting additional work by the author to proceed label Jun 23, 2026
@skarya22 skarya22 removed the State: Needs work PR awaiting additional work by the author to proceed label Jul 9, 2026
@skarya22
skarya22 self-requested a review July 9, 2026 15:04
@skarya22 skarya22 assigned skarya22 and unassigned Montekkundan Jul 9, 2026
@skarya22 skarya22 assigned Montekkundan and unassigned skarya22 Aug 31, 2026
@skarya22 skarya22 added the State: Needs rebase PR that needs to be rebased to proceed (conflicts, wrong branch...) label Aug 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language: PHP PR or issue that update PHP code Language: SQL PR or issue that update SQL code Module: configuration PR or issue related to configuration module RaisinBread PR or issue introducing/requiring improvements to the Raidinbread dataset State: Needs rebase PR that needs to be rebased to proceed (conflicts, wrong branch...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[configuration] Rework to support multidimensional UI

2 participants