-
Notifications
You must be signed in to change notification settings - Fork 70
GO-4969: REST API Stage 2 #2303
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
GO-4969: REST API Stage 2 #2303 8000 h1>
Conversation
Coverage provided by https://github.com/seriousben/go-patch-cover-action |
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.
Pull Request Overview
This PR implements "GO-4969: REST API Stage 2" by removing the legacy internal API endpoints and introducing a new set of REST endpoints under the handler package. Key changes include the complete removal of internal list, export, and auth packages and the addition of new endpoints for types, templates, tags, spaces, properties, objects, search, members, and list management.
Reviewed Changes
Copilot reviewed 79 out of 79 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
core/api/internal/list/model.go | Removed legacy list model definitions. |
core/api/internal/list/handler.go | Removed legacy list handlers. |
core/api/internal/export/model.go and handler.go | Removed legacy export endpoints. |
core/api/internal/auth/model.go and handler.go | Removed legacy authentication endpoints. |
core/api/handler/*.go (type.go, template.go, tag.go, space.go, search.go, property.go, object.go, member.go, list.go, auth.go) | Introduced new endpoints with updated Swagger annotations and restructured API handlers. |
core/api/core/mock_apicore/mock_ClientCommands.go | Updated mocks to support the new object type, relation, and option endpoint methods. |
core/api/core/core.go | Updated the interface to include new methods for object types, relations, and tag options. |
…etail handling
|
…error-handling GO-5591: Full-text: add missing error handling
New Endpoints
PATCH("/spaces/:space_id")
POST("/spaces/:space_id/types")
PATCH("/spaces/:space_id/types/:type_id")
DELETE("/spaces/:space_id/types/:type_id")
GET("/spaces/:space_id/properties")
GET("/spaces/:space_id/properties/:property_id")
POST("/spaces/:space_id/properties")
PATCH("/spaces/:space_id/properties/:property_id")
DELETE("/spaces/:space_id/properties/:property_id")
GET("/spaces/:space_id/properties/:property_id/tags")
GET("/spaces/:space_id/properties/:property_id/tags/:tag_id")
POST("/spaces/:space_id/properties/:property_id/tags")
PATCH("/spaces/:space_id/properties/:property_id/tags/:tag_id")
DELETE("/spaces/:space_id/properties/:property_id/tags/:tag_id")
Improvements and Fixes
id
alongside already returnedkey
properties
(featured and from sidebar) andplural_name
for typesIcon
,PropertyWithValue
andPropertyLinkWithValue
in swagger specBreaking Changes
recommended_layout
tolayout
for typesdescription
andsource
to[]properties
inCreateObjectRequest
todo
toaction
for consistencyot-
prefix for type keysproperty
toproperty_key
in sort options in searchblocks
in getObject in favor ofmarkdown
exported body (proper json block export will be added later)