Skip to content

[PATCH] GovernanceWidget follow-up: skip onboarding, Storybook alignment, QA re-audit #172

Description

@goodbounties-nanoclaw-agent

Summary

Follow-up to #77 (GovernanceWidget runtime wired to GoodDaoHouses). #80's requested changes are already merged into copilot/plan-wire-governancewidget-to-gooddaohouses via #99. This tracks the next round of follow-up items requested in #gd-patch, to be delivered as a new branch/PR stacked on copilot/plan-wire-governancewidget-to-gooddaohouses (not main).

Scope

Onboarding skip

  • Add a "Skip for now" control on the onboarding_required state that lets a connected user reach the read-only dashboard (GovernanceDashboard) without completing signup. All actionable controls stay gated behind active-member status as they already are.
  • Keep a persistent "Sign up and stake to participate in GoodDAO" banner/CTA visible while skipped, which reopens onboarding.
  • Skip state is local/in-memory only — resets on reload or reconnect (no persistence across sessions).

Storybook alignment (widgets section)

  • Add a GovernanceWidget.mdx docs page, following the existing convention used by other widgets (e.g. StakingMigrationWidget.mdx, AiCreditsWidget.mdx).
  • Keep the live/injected-wallet Showcase story live-only (real contract reads) — no mock/live toggle mixed into it.
  • Add a new "live testable flow with mocked data" story under the QA section: an interactive walk-through of the flow using mocked adapter data, kept separate from the existing static QA fixture snapshots and from the live Showcase story.
  • Add a GovernanceWidgetThemeOverrides.stories.tsx, following StakingMigrationWidgetThemeOverrides.stories.tsx (live color-picker controls + generated code snippet), using the existing light_/dark_ theme targets in config.ts.

QA section

Contract alignment

  • Re-verify runtime reads/writes (getMember, getVoteConfig, getHoaEligibility, unstake, funding reads, etc.) against the deployed dev-celo GoodDaoHouses contract at 0x4Bc3Cdc036f21b68E034C0f1d90775fc3D725735. ABI is unchanged since GoodProtocol#299 (merged); GoodProtocol#300 (dev deployment record) is still open but the address already has live bytecode on Celo.
  • If anything is found missing from the deployed contract, raise it as a PR comment rather than adding a workaround.

Out of scope

References

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    Status
    Prepare AI Task

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions