-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Updated RESTful API Reference #908
Conversation
Validated and improved clarity of the API reference. I may have tempered with the intended meaning, please double check. |
There is still room for improvement, for example, descriptions of the API behavior are still on the thin side. But I feel the current version is ready for initial publication. |
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
Add DEMO link at the first paragraph of README |
docs/references/api.md
Outdated
|
||
## Delete document | ||
- Path: `/api/document` |
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.
Path is also not a very proper word. Change it to: Request URI
### What problem does this PR solve? Update docnm_kwd to doc_name #908 ### Type of change - [x] Refactoring
### What problem does this PR solve? _Briefly describe what this PR aims to solve. Include background context that will help reviewers understand the purpose of the PR._ ### Type of change - [x] Documentation Update
### What problem does this PR solve? Update docnm_kwd to doc_name infiniflow#908 ### Type of change - [x] Refactoring
What problem does this PR solve?
Briefly describe what this PR aims to solve. Include background context that will help reviewers understand the purpose of the PR.
Type of change