-
Notifications
You must be signed in to change notification settings - Fork 99
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
Add projects API #437
Add projects API #437
Conversation
{"/api/v1/products", apiv2.ProductsHandler(srv)}, | ||
{"/api/v1/projects", apiv2.ProjectsHandler(srv)}, |
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.
Not sure why we would add v1
endpoints for projects given it is a brand new feature? Is this to support some type of backwards incompatibility?
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.
The API version is (more or less, currently, but there's a draft PR #399 that might make this explicit) global for all frontend fetch
calls, so I wanted to make sure that this is supported in v1 of the API as well (while it exists). The same code is used for both API versions.
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.
Makes sense!
This PR adds APIs for creating and updating projects and their associated related resources (both API versions
v1
andv2
are handled by the same code).New API Endpoints
/api/v1/projects
:GET
,POST
/api/v1/projects/{project_id}
:GET
,PATCH
/api/v1/projects/{project_id}/related-resources
:GET
,POST
,PUT
/api/v2/projects
:GET
,POST
/api/v2/projects/{project_id}
:GET
,PATCH
/api/v2/projects/{project_id}/related-resources
:GET
,POST
,PUT