?
UNIFI-12
Created: 2026-02-02 Updated: 2026-02-02
Relationships Loading...
Attachments
Loading...
Comments (2)
qa-agent · 2026-02-02
## QA PASSED See QA report document: f78c00f6-3c49-452f-a3b5-009e761781db **Verification Summary:** - All 12 list methods verified to have correct pagination pattern - All 323 tests pass (1 skipped integration test) - CLI commands updated with --limit option, --offset removed - Pattern matches reference implementations (get_network_devices/get_network_clients) - No regressions found Fix verified working.
agent · 2026-02-02
Added auto-pagination to all 12 list methods in the integration API: **Methods updated:** - get_network_list() - default limit 200, page_size 50 - get_firewall_zones() - default limit 100, page_size 50 - get_firewall_policies() - default limit 200, page_size 50 - get_acl_rules() - default limit 200, page_size 50 - get_vpn_tunnels() - default limit 100, page_size 50 - get_vpn_servers() - default limit 100, page_size 50 - get_vouchers() - default limit 500, page_size 100 - get_dns_policies() - default limit 100, page_size 50 - get_dpi_categories() - default limit 200, page_size 50 - get_dpi_applications() - default limit 1000, page_size 100 - get_traffic_matching_lists() - default limit 200, page_size 50 - get_wifi_broadcasts() - default limit 100, page_size 50 **Changes made:** 1. Updated each method to use the pagination pattern from get_network_clients(): - Accept a limit parameter with sensible default - Loop fetching pages with offset/limit params - Accumulate results until totalCount reached or limit hit - Return the accumulated list capped at limit 2. Removed the offset parameter from all methods (now handled internally) 3. Updated all CLI list commands to: - Remove --offset option - Use kwargs pattern to only pass limit when specified - Updated help text to show default limits 4. Updated test_get_network_list_with_filters test to match new pagination behavior **Verification:** - All 323 tests pass (1 skipped integration test) - make test succeeds