Skip to content

Network-first pools management and default networks feature#75

Closed
fengtality wants to merge 1 commit intomainfrom
feat/network-pools-management
Closed

Network-first pools management and default networks feature#75
fengtality wants to merge 1 commit intomainfrom
feat/network-pools-management

Conversation

@fengtality
Copy link
Copy Markdown
Contributor

Summary

  • Refactored pools management to use network-first approach (like tokens menu)
  • Added default networks feature: users can toggle networks as defaults in the Networks menu
  • Tokens and Pools menus now show only default networks with an "All Networks" button to see all
  • Added checkmark indicator for default networks in the Networks menu
  • Removed unnecessary "Restart Gateway" messages from pools (adding/removing pools doesn't require restart)

Changes

  • handlers/config/gateway/pools.py - Complete refactor to network-first approach with pagination
  • handlers/config/gateway/tokens.py - Added default networks filtering with "All Networks" option
  • handlers/config/gateway/networks.py - Added default network toggle and checkmark display
  • handlers/config/gateway/_shared.py - Added get_default_networks() helper function
  • mcp_servers/hummingbot_api/tools/gateway.py - Updated pool handlers to use network-based API

Test plan

  • Verify Networks menu shows checkmarks on default networks
  • Verify toggling default status works correctly
  • Verify Tokens menu shows only default networks initially
  • Verify "All Networks" button shows all networks in Tokens menu
  • Verify Pools menu shows only default networks initially
  • Verify "All Networks" button shows all networks in Pools menu
  • Verify pool pagination works correctly
  • Verify adding/removing pools works without restart messages

🤖 Generated with Claude Code

@fengtality fengtality closed this Apr 28, 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.

1 participant