Skip to content

Render nested DAO proposal details - #1559

Merged
osyed merged 3 commits into
mainfrom
phase-5-dao-render-proposal-details
Aug 12, 2026
Merged

Render nested DAO proposal details#1559
osyed merged 3 commits into
mainfrom
phase-5-dao-render-proposal-details

Conversation

@dnlbui

@dnlbui dnlbui commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Render every proposal option with its matching nested parameter-change set.
  • Use the same option renderer in the proposal review and proposal-info modals.
  • Rename the DAO repository module from dao.repo.js to dao.js, including application and documentation references.
  • Update DAO documentation and modal styles for the nested presentation.

Why

Nested DAO proposals associate each action option with a positional change set, so proposal details need to show those values together. The module rename gives the DAO data boundary a simpler filename.

Validation

  • node --check app.js
  • node --experimental-default-type=module --test tests/dao-proposal-create.test.mjs tests/dao-summary-fetcher.test.mjs tests/dao-timeline.test.mjs

@dnlbui
dnlbui marked this pull request as ready for review August 10, 2026 22:07
@dnlbui dnlbui linked an issue Aug 10, 2026 that may be closed by this pull request
@dnlbui
dnlbui marked this pull request as draft August 10, 2026 22:10
@dnlbui
dnlbui marked this pull request as ready for review August 10, 2026 22:10
@dnlbui dnlbui linked an issue Aug 10, 2026 that may be closed by this pull request
5 tasks
@osyed
osyed force-pushed the phase-5-dao-render-proposal-details branch from f6a71fb to c4fa19b Compare August 11, 2026 15:03
@osyed
osyed force-pushed the phase-5-dao-render-proposal-details branch from c4fa19b to 20ecbf4 Compare August 11, 2026 15:09
Base automatically changed from phase-4-dao-submit-nested-proposals to main August 12, 2026 14:28
@osyed
osyed force-pushed the phase-5-dao-render-proposal-details branch from 20ecbf4 to 20616c9 Compare August 12, 2026 14:28
@osyed
osyed merged commit 430a11d into main Aug 12, 2026
1 check 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.

DAO: render multi-option proposal change sets in proposal details Parent: Update DAO UI for multi-option proposals and summary index

2 participants