bramhanandlingala commented on PR #9049: URL: https://github.com/apache/devlake/pull/9049#issuecomment-5296170931
@anthonyhull-ford Thanks for looking into this closely — good catch. You're right that this doesn't cover centrally-licensed seats. I checked GitHub's docs, and when a seat is licensed through multiple orgs in the same enterprise, GitHub bills (and reports) only one org per seat, chosen by GitHub — so the seats API itself never exposes the user's other org memberships. That means Product Org A and B could still show zero denominators in that case, but it's not fixable from this endpoint alone; it'd need a separate join against org membership data. I'd like to keep that as a follow-up rather than block this fix, since it's a different data source and a bigger scope. Also — noticed the CI checks aren't triggering on this PR. Could someone approve the workflow run? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
