chore(docs): add openAPI schema#120
Conversation
|
As commented in Slack: URL2M:
RAG
|
nicklamonov
left a comment
There was a problem hiding this comment.
Thanks! All good from my side.
nikitachapovskii-dev
left a comment
There was a problem hiding this comment.
Thank you, @ruocco-l !
I noticed rag readme is updated for openapi but url2m not, maybe we should update it as well?
There was a problem hiding this comment.
Did you skip some fields because they are hidden? (removeElementsCssSelector, desiredConcurrency, debugMode)
Should we in that case skip requestTimeoutSecs and debugMode for rag?
There was a problem hiding this comment.
Yeah we can do that
| "description": "Apify Proxy configuration used for scraping the target web pages.", | ||
| "required": false, | ||
| "schema": { | ||
| "type": "object", |
There was a problem hiding this comment.
I asked Claude to compare this PR with the web-fetch one (apify/actor-web-fetch#1) and it flagged a difference that proxyConfiguration here has type: object while there they got headers type: string (JSON-encoded) for the GET
I guess we should align it here to be string as well so it JSON.parse's correclty
{
"name": "headers",
"in": "query",
"required": false,
"schema": {
"type": "string",
"description": "Additional HTTP headers to send with the request, as a JSON-encoded object string."
},
"example": "{\"Accept-Language\":\"de-DE\"}"
},
There was a problem hiding this comment.
Yeah, this is correct, I'll change it
Closes #92
Adding openAPI for both RAG and URL2M