Skip to content

Unify campaign/investor pages on soil/leaf tokens and wire CampaignDetailPage to :id - #197

Open
matthewrball wants to merge 3 commits into
Cylo-Traders:masterfrom
matthewrball:fix/issues-148-186-design-and-route
Open

Unify campaign/investor pages on soil/leaf tokens and wire CampaignDetailPage to :id#197
matthewrball wants to merge 3 commits into
Cylo-Traders:masterfrom
matthewrball:fix/issues-148-186-design-and-route

Conversation

@matthewrball

Copy link
Copy Markdown
Contributor

Summary

Closes #148
Closes #186

Notes

  • Shared Card/Button CSS modules are still the old slate HSL system. These pages use Tailwind tokens directly so they match Header/StatTile.
  • Pre-existing on master (not in this PR): ActivityFeed.test.tsx failures and an ImportMeta tsc error in ActivityFeedPage.
  • GrantFox Third Campaign listing: Maybe Rewarded / not escrowed. Happy to apply/assign if that's the preferred claim path.

Test plan

  • CampaignDetailPage with a real :id loads that campaign (or an error if missing)
  • /campaigns and /campaigns/new still resolve
  • Investor dashboard + investment cards use soil/leaf tokens, not slate/emerald
  • Status badges and fund CTA match Header/StatTile contrast
  • New CampaignDetailPage tests pass

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.

CampaignDetailPage ignores its route param and always renders mock data Two incompatible design systems are mixed across the app

1 participant