Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Jun 16, 2025

Important

Fetch and display organization information in the extension by updating AuthService, CloudService, and AccountView components.

  • AuthService:
    • Adds clerkGetOrganizationMemberships() to fetch organization memberships.
    • Updates clerkMe() to include organization info in CloudUserInfo.
  • CloudService:
    • Adds getOrganizationId(), getOrganizationName(), and getOrganizationRole() to access organization info.
  • Types:
    • Adds organizationId, organizationName, and organizationRole to CloudUserInfo in cloud.ts.
  • UI:
    • Updates AccountView.tsx to display organizationName if available.
  • Tests:
    • Adds tests in CloudService.test.ts for organization info methods.

This description was created by Ellipsis for 68d0254. You can customize this summary. It will automatically update as commits are pushed.

@mrubens mrubens requested review from cte and jr as code owners June 16, 2025 18:30
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 16, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 16, 2025
@cte
Copy link
Collaborator

cte commented Jun 16, 2025

Should we try and make this more vendor agnostic by dropping the references to Clerk?

@mrubens
Copy link
Collaborator Author

mrubens commented Jun 16, 2025

Should we try and make this more vendor agnostic by dropping the references to Clerk?

Maybe we can discuss with @jr as a follow-up? I think Roo was just following the other examples in this file.

@mrubens
Copy link
Collaborator Author

mrubens commented Jun 16, 2025

Maybe the types should be more generic though?

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 16, 2025
@mrubens mrubens merged commit 322a15e into main Jun 16, 2025
9 checks passed
@mrubens mrubens deleted the organization_info branch June 16, 2025 18:58
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 16, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 16, 2025
shivangag pushed a commit to IdenWorks/Roo-Code that referenced this pull request Jun 17, 2025
valekseev pushed a commit to valekseev/Roo-Code that referenced this pull request Jun 18, 2025
cte pushed a commit that referenced this pull request Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants