Skip to content

fix: Include sub-plugins when fetching plugins on the member details page#1047

Merged
milosh86 merged 1 commit intomainfrom
member-details-hot-fix
Mar 21, 2026
Merged

fix: Include sub-plugins when fetching plugins on the member details page#1047
milosh86 merged 1 commit intomainfrom
member-details-hot-fix

Conversation

@milosh86
Copy link
Contributor

@milosh86 milosh86 commented Mar 21, 2026

Description

Adds includeSubPlugins: true to the useDaoPlugins call in daoMemberDetailsPageClient. Without this flag, SPP sub-plugins (used for vote submission, member details, and version info) were not included, causing the member details page to not resolve the correct body plugin when the DAO uses a Staged Proposal Pipeline.

Type of Change

  • Patch: Bug fix (non-breaking change which fixes an issue)

Developer Checklist:

  • Manually smoke tested the functionality in a preview or locally
  • Confirmed there are no new warnings or errors in the browser console
  • (For User Stories only) Double-checked that all Acceptance Criteria are satisfied
  • Confirmed there are no new warnings on automated tests
  • Merged and published any dependent changes in downstream modules
  • Selected the correct base branch
  • Commented the code in hard-to-understand areas
  • Followed the code style guidelines of this project
  • Reviewed that the Files Changed in Github's UI reflect my intended changes
  • Confirmed the pipeline checks are not failing

Review Checklist:

  • (For User Stories only) Tested in a preview or locally that all Acceptance Criteria are satisfied
  • Confirmed that changes follow the code style guidelines of this project

…page

Signed-off-by: Milos Dzepina <milos@aragon.org>
@milosh86 milosh86 self-assigned this Mar 21, 2026
@milosh86 milosh86 marked this pull request as ready for review March 21, 2026 17:11
@milosh86 milosh86 requested a review from a team as a code owner March 21, 2026 17:11
Copilot AI review requested due to automatic review settings March 21, 2026 17:11
Copy link
Contributor

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

Fixes DAO member details rendering for DAOs using Staged Proposal Pipeline (SPP) by ensuring sub-plugins are included when resolving the “body” plugin metadata on the client.

Changes:

  • Enable includeSubPlugins: true in the useDaoPlugins call on the member details client page.
  • Add .patches to .gitignore.
  • Add a Changesets patch entry for the release notes/versioning.

Reviewed changes

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

File Description
src/modules/governance/pages/daoMemberDetailsPage/daoMemberDetailsPageClient.tsx Ensures sub-plugins are included so the correct body plugin can be resolved under SPP.
.gitignore Ignores a .patches directory (needs confirmation it shouldn’t be committed).
.changeset/member-details-include-sub-plugins.md Adds a patch changeset documenting the fix.

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

@github-actions
Copy link

🚀 Preview Deployment: View Here

@github-actions
Copy link

E2E Smoke Results

Base URL https://app-next-1ascl91hr-aragon-app.vercel.app
Suite smoke
Result ✅ passed
Run #3896

@milosh86 milosh86 merged commit 73df695 into main Mar 21, 2026
22 checks passed
@milosh86 milosh86 deleted the member-details-hot-fix branch March 21, 2026 19:07
milosh86 added a commit that referenced this pull request Mar 21, 2026
…page (#1047)

Signed-off-by: Milos Dzepina <milos@aragon.org>
(cherry picked from commit 73df695)
@arabot-1 arabot-1 mentioned this pull request Mar 21, 2026
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.

3 participants