←
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