-
Notifications
You must be signed in to change notification settings - Fork 141
docs: replace all api docs links #1690
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
Conversation
replace all old api-docs link to current working URLs fix prose
|
Preview for this PR was built for commit |
tomnosek
left a comment
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.
I wasn't checking the exactness of the links, but I presume a link checker would find insufficiencies.
| --- | ||
|
|
||
| [Apify's API](/api/v2#/reference) is your ticket to the Apify platform without even needing to access the [Apify Console](https://console.apify.com?asrc=developers_portal) web-interface. The API is organized around RESTful HTTP endpoints. | ||
| [Apify's API](/api/v2) web-interface. The API is organized around RESTful HTTP endpoints. |
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.
It seems there was an editing mistake here - removing up to the wrong parenthesis.
| With that, we're done! And our application works like a charm :) | ||
|
|
||
| The complete code of this Actor is available [here](https://apify.com/apify/example-web-server). You can run it there or copy it to your account. | ||
| The complete code of this Actor is available at it's [page](https://apify.com/apify/example-web-server). You can run it there or copy it to your account. |
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.
It should be "on its", not "at it's". But also, saying "on its page" with a basic link is lazy writing 😄 Why not to say "on its Apify Store page" and link it directly to https://apify.com/apify/example-web-server/source-code?
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.
That is what I get for making last minute fixes late at night , good catch will address it
|
Yeah, link checker should have caught anything that is not working. This is just fix to not use legacy URLs that seem to have broke and are not working correctly when trying to open them, so I swapped them with current URLs and by checking on preview it seems to point to correct endpoints and opens up correctly now , no matter whether you try to open it on new tab or within the same tab |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
replace all old api-docs link to current working URLs
fix prose