Skip to content

Feature/GitHub remove org membership to free seat#65

Merged
betterclever merged 8 commits intomainfrom
feature/GitHub-remove-org-membership-to-free-seat
Nov 6, 2025
Merged

Feature/GitHub remove org membership to free seat#65
betterclever merged 8 commits intomainfrom
feature/GitHub-remove-org-membership-to-free-seat

Conversation

@AshhKetchup
Copy link
Contributor

Summary

  • Introduces reusable 3rd‑party connection flow and a GitHub Connection Provider to pass connectionId downstream; worker retrieves tokens via POST /integrations/connections/:id/token guarded by INTERNAL_SERVICE_TOKEN (no tokens in UI).

  • Adds GitHub Remove Org Membership component to optionally remove from a team then the org; resolves email→username, emits progress, and treats 404s as idempotent “not found”.

  • Enables secure, repeatable GitHub offboarding/seat reclamation using a single OAuth connection shared across components.

Touched files: worker/src/components/github/connection-provider.ts, worker/src/components/github/remove-org-membership.ts, worker/src/components/index.ts, backend/src/integrations/integrations.controller.ts, backend/src/integrations/integrations.dto.ts

Testing

bun run test
bun run lint
bun run typecheck
Additional notes: E2E sanity — wire “GitHub Connection Provider” → “GitHub Remove Org Membership”; set organization, optional teamSlug, userIdentifier; ensure a GitHub connection exists; minimal scopes: admin:org read:org.

@AshhKetchup AshhKetchup force-pushed the feature/GitHub-remove-org-membership-to-free-seat branch from 9528d41 to 1e78082 Compare November 5, 2025 15:52
@betterclever betterclever force-pushed the feature/GitHub-remove-org-membership-to-free-seat branch from 66b4c70 to 427ac07 Compare November 6, 2025 19:55
@betterclever betterclever merged commit bcec86e into main Nov 6, 2025
1 check passed
@LuD1161 LuD1161 deleted the feature/GitHub-remove-org-membership-to-free-seat branch December 19, 2025 13:21
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