Skip to content

Add ExportAccessControlCounts to StartExportFlow in generated schema - #173

Merged
rmennes merged 1 commit into
mainfrom
read-out-export-count-in-sdk
Nov 9, 2026
Merged

rmennes merged 1 commit into
mainfrom
read-out-export-count-in-sdk

Conversation

@rmennes

@rmennes rmennes commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

📝 Description of the changes

Adds access control counts (grants, groups, shares, masks, filters) to the export flow's StartExportFlow response, so clients can see how many access control entries an export carries when it starts. The GraphQL query, its generated Go types, and the public types aliases are updated together to keep the client in sync with the schema.

Export Flow API

  • Extended the StartExportFlow fragment to request accessControlCounts, reusing a new ExportAccessControlCounts fragment (internal/schema/queries/exporter.graphql).
  • Added the ExportAccessControlCounts struct with getters for each count field, and wired it into StartExportFlow and TriggerExportFlowStartExportFlow (internal/schema/generated.go).
  • Updated the TriggerExportFlow query string to include the new selection and fragment (internal/schema/generated.go).

Generated Types

  • Exposed ExportAccessControlCounts and StartExportFlowAccessControlCountsExportAccessControlCounts as type aliases in the public types package (types/generated.go).

Impact & Risk

  • The TriggerExportFlow response now includes an additional accessControlCounts object. Clients that decode strictly or compare full payloads may need to account for it.
  • The diff is truncated, so changes after types/generated.go were not reviewed.

🎫 JIRA reference

No JIRA ticket detected.

@rmennes
rmennes requested a review from a team as a code owner November 9, 2026 14:40
@rmennes
rmennes enabled auto-merge November 9, 2026 14:43
@sonarqube-collibra

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@rmennes
rmennes merged commit 58584e0 into main Nov 9, 2026
5 checks passed
@rmennes
rmennes deleted the read-out-export-count-in-sdk branch November 9, 2026 14:49
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