-
Notifications
You must be signed in to change notification settings - Fork 3.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃帀 API: Add an endpoint to list workspaces #4530
馃帀 API: Add an endpoint to list workspaces #4530
Conversation
Thanks @mmolimar I'm going to review later today! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mmolimar looks great! thanks for adding it. Just a small request for unit tests and we should be ready to go 馃帀
@@ -141,6 +144,13 @@ public void deleteWorkspace(final WorkspaceIdRequestBody workspaceIdRequestBody) | |||
configRepository.writeStandardWorkspace(persistedWorkspace); | |||
} | |||
|
|||
public WorkspaceReadList listWorkspaces() throws JsonValidationException, IOException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add unit test to verify this? You can add in WorkspacesHandlerTest.java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure! I just added it @sherifnada ;-)
Thanks @mmolimar ! 馃帀 |
What
This PR adds a new endpoint in the API to list all workspaces. Related with #4003
How
Update the YAML file containing the endpoints with the specification and the WorkspaceHandler to enable listing the workspaces.
Recommended reading order
config.yml
ConfigurationApi.java
WorkspacesHandler.java
Pre-merge Checklist
Expand the checklist which is relevant for this PR.
Connector checklist
airbyte_secret
in the connector's spec./gradlew :airbyte-integrations:connectors:<name>:integrationTest
./test connector=connectors/<name>
command as documented here is passing.README.md
docs/SUMMARY.md
if it's a new connectordocs/integrations/<source or destination>/<name>
.docs/integrations/...
. See changelog exampledocs/integrations/README.md
contains a reference to the new connector/publish
command described hereConnector Generator checklist
-scaffold
in their name) have been updated with the latest scaffold by running./gradlew :airbyte-integrations:connector-templates:generator:testScaffoldTemplates
then checking in your changes