Skip to content

Wire up organization client endpoints into OrganizationsClientsSection#324

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/add-search-functionality-for-clients
Draft

Wire up organization client endpoints into OrganizationsClientsSection#324
Copilot wants to merge 2 commits intomainfrom
copilot/add-search-functionality-for-clients

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 4, 2026

IOrganizationsClientOverview and IOrganizationBandwidthUsageHistory interfaces existed but were never wired into MerakiClient or OrganizationsSection. The IOrganizationsClients (search) was wired as a flat interface rather than a proper section.

Changes

  • New OrganizationsClientsSection — Groups all three client interfaces following the NetworksClientsSection pattern. Uses [RefitPromoteCalls] on the internal IOrganizationsClients for backward compatibility.
  • OrganizationsSection.Clients — Changed from IOrganizationsClients to OrganizationsClientsSection
  • MerakiClient wiring — Wires up Clients, BandwidthUsageHistory, and Overview

Existing consumer code is unaffected:

// Still works — promoted via source generator
client.Organizations.Clients.GetOrganizationClientsSearchAsync(orgId, mac);

// Now also available
client.Organizations.Clients.Overview.GetOrganizationClientsOverviewAsync(orgId, t0, t1);
client.Organizations.Clients.BandwidthUsageHistory.GetOrganizationClientsBandwidthUsageHistoryAsync(orgId, t0, t1);

@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Apr 4, 2026

Not up to standards ⛔

🔴 Issues 1 minor

Alerts:
⚠ 1 issue (≤ 0 issues of at least minor severity)

Results:
1 new issue

Category Results
CodeStyle 1 minor

View in Codacy

AI Reviewer: run a review on demand. To trigger the first review automatically, go to your organization or repository integration settings. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes. Give us feedback

…tory endpoints

Create OrganizationsClientsSection to properly group all organization-client
interfaces (IOrganizationsClients, IOrganizationsClientOverview, and
IOrganizationBandwidthUsageHistory) following the established section pattern.

- Add OrganizationsClientsSection.cs with RefitPromoteCalls for backward compat
- Update OrganizationsSection.cs Clients property to use new section type
- Update MerakiClient.cs to wire up all three client interfaces

Agent-Logs-Url: https://github.com/panoramicdata/Meraki.Api/sessions/20dfd578-4d0e-48ec-ab9d-98478d1a1b16

Co-authored-by: danielabbatt <8926756+danielabbatt@users.noreply.github.com>
Copilot AI changed the title [WIP] Add endpoint for searching organization clients Wire up organization client endpoints into OrganizationsClientsSection Apr 4, 2026
Copilot AI requested a review from danielabbatt April 4, 2026 17:38
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.

Unable to search all organization clients

2 participants