Skip to content

Conversation

@pfurio
Copy link
Member

@pfurio pfurio commented Sep 22, 2025

No description provided.

@pfurio pfurio requested a review from juanrizetta September 22, 2025 09:53
@halender
Copy link
Contributor

@j-coll j-coll changed the title Fix: update JWT secret key for new environments TASK-7922 - Fix: update JWT secret key for new environments Sep 22, 2025
@juanrizetta juanrizetta requested a review from Copilot October 21, 2025 10:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a new API endpoint to list current organizations across all client implementations (Java, Python, JavaScript, R) and the CLI tool. While the PR title references JWT secret key updates, the actual changes implement organization listing functionality.

Key Changes:

  • Added /admin/organizations/list GET endpoint in the server
  • Implemented corresponding client methods across all supported languages
  • Added CLI command organizations-list for command-line access

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
AdminWSServer.java Adds new GET endpoint /admin/organizations/list that returns organization IDs
admin_client.py Implements Python client method list_organizations()
Admin.js Implements JavaScript client method listOrganizations()
AdminClient.java Implements Java client method listOrganizations()
Admin-methods.R Implements R client method for organization listing
AdminCommandOptions.java Adds CLI command options class for organizations-list command
AdminCommandExecutor.java Implements CLI executor for organization listing
OpencgaCliOptionsParser.java Registers the new organizations-list subcommand
OpenCgaCompleter.java Adds autocomplete support for the new command

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants