-
Notifications
You must be signed in to change notification settings - Fork 17
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: (APIC-190) Add manage indices endpoints #24
Conversation
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.
3 more ! I have a few comments.
(We should wait for #21 before merging this one) |
clients/algoliasearch-client-javascript/client-search/src/searchApi.ts
Outdated
Show resolved
Hide resolved
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.
Small format comments
properties: | ||
taskID: | ||
$ref: '../../../common/parameters.yml#/taskID' | ||
updatedAt: | ||
$ref: '../../../common/parameters.yml#/updatedAt' |
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.
Is there a way to make those response properties not optional? I see that we have that everywhere, while it shouldn't be the case
(Shouldn't be fixed in this PR IMO since it needs to be applied on every responses)
clients/algoliasearch-client-javascript/client-search/model/listIndicesResponse.ts
Show resolved
Hide resolved
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.
perfecto
Added :
/1/indexes/{indexName}/operation
(copy/move index) : https://www.algolia.com/doc/rest-api/search/#copymove-index/1/indexes
(list indices): https://www.algolia.com/doc/rest-api/search/#list-indices/1/indexes/{indexName}
(delete method from the already existing search endpoint) (delete index) : https://www.algolia.com/doc/rest-api/search/#delete-index