Skip to content

feat: add GDA pool management pages and pool member connection#199

Merged
tnrdd merged 13 commits intomainfrom
orgin/feat/tryConnectFor
Mar 23, 2026
Merged

feat: add GDA pool management pages and pool member connection#199
tnrdd merged 13 commits intomainfrom
orgin/feat/tryConnectFor

Conversation

@gravenp
Copy link
Copy Markdown
Member

@gravenp gravenp commented Feb 14, 2026

Summary

  • GDA distribution pool pages — new /pools listing, detail, and admin views for managing GDA distribution pools with pool member tables, flow rate configuration, and instant distributions
  • Pool member connectiontryConnectPoolFor button on flow splitter pool graph nodes, batch "Connect All" on pool nodes with status overlays, automatic chain switching, and wallet connect modal for unauthenticated users
  • UI polish — consistent button text sizing and card border rounding across listing pages, clickable pool cards on flow splitters page

Test plan

  • Verify /pools listing page loads and displays GDA pools
  • Verify pool detail page shows member list and pool info
  • Verify pool admin page allows flow rate and distribution management
  • Test tryConnectPoolFor on a disconnected member in pool graph
  • Test Connect All button connects multiple disconnected members
  • Test wallet connect modal appears for unauthenticated users
  • Test chain switching when connected to wrong network
  • Confirm no regressions on flow splitters and project pages

🤖 Generated with Claude Code

Allow users to connect disconnected pool members via the GDA's
tryConnectPoolFor function called through Host.callAgreement. Adds
GDA agreement addresses per network, minimal ABIs, transaction status
overlay on member avatars, and post-tx isMemberConnected verification.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
platform Ready Ready Preview, Comment Mar 23, 2026 3:35am

Request Review

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Make flow splitter and flow council cards fully clickable, navigating
to the view/voting page. Fix PoolConnectionButton missing fs-lg class
to match Edit and View button text size.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove fs-lg from flow splitter Edit/View buttons and PoolConnectionButton
to match flow council styling. Change flow splitter pool cards from
rounded-5 to rounded-4 to match the Create card.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Uses Host.batchCall to connect all disconnected pool members in a single transaction.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…onnect All or tryConnectPoolFor

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add /pools route for viewing and managing any Superfluid GDA distribution
pool by address. Includes landing page with on-chain validation, detail
view with PoolGraph/OpenFlow/InstantDistribution, and admin page with
batch updateMemberUnits via host.batchCall.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gravenp gravenp changed the title feat: add tryConnectPoolFor button to flow splitter pool graph feat: add GDA pool management pages, pool member connection, and codebase cleanup Mar 21, 2026
@gravenp gravenp changed the title feat: add GDA pool management pages, pool member connection, and codebase cleanup feat: add GDA pool management pages and pool member connection Mar 21, 2026
… name magic number

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
gravenp and others added 2 commits March 20, 2026 19:35
Replace shallow-copy-then-mutate pattern with .map() to create new
objects for modified entries. Also replace splice-during-iteration
in CSV upload with .filter(). Applies to both pool-admin and
flow-splitters admin pages.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
… text

Add comment explaining why handleConnectAll skips post-tx member
verification (N RPC calls, slot exhaustion is rare). Simplify
redundant Connect All button text ternary.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gravenp
Copy link
Copy Markdown
Member Author

gravenp commented Mar 21, 2026

@tnrdd I checked and cleaned this one up. Ready for review.

The new Superfluid SDK will be integrated separately.

}
poolDistributors(first: 1000, where: { flowRate_not: "0" }) {
account {
id
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: unitsTrasnferabilityunitsTransferability (missing the i).

() =>
membersEntry
.map((memberEntry) =>
isNumber(memberEntry.units) ? Number(memberEntry.units) : 0,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This async IIFE wraps purely synchronous code — it can be a plain function body.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@tnrdd tnrdd merged commit 68da918 into main Mar 23, 2026
2 checks passed
@tnrdd tnrdd deleted the orgin/feat/tryConnectFor branch March 23, 2026 05:52
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.

2 participants