Skip to content

[configuration] Show nested configuration settings - #10606

Merged
driusan merged 4 commits into
aces:mainfrom
Montekkundan:montek/10471-config-tree-ui-clean
Oct 27, 2026
Merged

driusan merged 4 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
@driusan

driusan commented Oct 6, 2026

Copy link
Copy Markdown
Collaborator

@Montekkundan This has conflicts and needs to be rebased

@Montekkundan
Montekkundan force-pushed the montek/10471-config-tree-ui-clean branch from c7b9e4c to e572c6b Compare October 6, 2026 18:30
@driusan

driusan commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator

@Montekkundan Conflicts again after merging your last PR

@driusan

driusan commented Oct 26, 2026

Copy link
Copy Markdown
Collaborator

@Montekkundan conflicts

@driusan
driusan merged commit a40ffe7 into aces:main Oct 27, 2026
22 checks passed
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

3 participants