-
Notifications
You must be signed in to change notification settings - Fork 2
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/v2 workspaces #96
Conversation
… into feat/v2_workspaces
… into feat/v2_workspaces
New feature: Hooks can have multiple associated project_ids |
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.
Only some superficial comments that can be addressed before merge.
@@ -234,6 +234,8 @@ message UpdateObjectRequest { | |||
string dataset_id = 10; | |||
} | |||
repeated storage.models.v2.Hash hashes = 12; | |||
// Force new object revision | |||
bool forced_revision = 13; |
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.
I think force_revision
sounds better for a request field.
bool forced_revision = 13; | |
bool force_revision = 13; |
@@ -23,6 +22,42 @@ service WorkspaceService { | |||
body : "*" | |||
}; | |||
} | |||
// ListWorkspaceTemplatesById |
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.
Documentation should be changed also.
// ListWorkspaceTemplatesById | |
// GetWorkspaceTemplate |
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.
lgtm
Workspaces
Hooks