-
Notifications
You must be signed in to change notification settings - Fork 135
feat: Minor updates #1726
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: Minor updates #1726
Conversation
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
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.
Comment bugbot run to trigger another review on this PR
|
Preview for this PR was built for commit |
sources/platform/actors/index.mdx
Outdated
|
|
||
| ## Running Actors | ||
|
|
||
| You can run Actors manually in [Apify Console](https://console.apify.com/actors), using the [API](/api), [CLI](/cli), or [scheduler](../schedules.md). You can easily [integrate Actors](../integrations/index.mdx) with other apps, [share](./collaboration/access-rights) them with other people, [publish](../actors/publishing/index.mdx) them in [Apify Store](https://apify.com/store), and even [monetize](../actors/publishing/monetize/index.mdx). |
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.
| You can run Actors manually in [Apify Console](https://console.apify.com/actors), using the [API](/api), [CLI](/cli), or [scheduler](../schedules.md). You can easily [integrate Actors](../integrations/index.mdx) with other apps, [share](./collaboration/access-rights) them with other people, [publish](../actors/publishing/index.mdx) them in [Apify Store](https://apify.com/store), and even [monetize](../actors/publishing/monetize/index.mdx). | |
| You can run Actors manually in [Apify Console](https://console.apify.com/actors), using the [API](/api), [CLI](/cli), or [scheduler](../schedules). You can [integrate](../integrations/index.mdx) them with other apps, [share ](./collaboration/access-rights) them with others, [publish ](../actors/publishing) them in [Apify Store](https://apify.com/store), and even [monetize ](../actors/publishing/monetize) them. |
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
Co-authored-by: Patrik Braborec <[email protected]>
|
Preview for this PR was built for commit |
1 similar comment
|
Preview for this PR was built for commit |
|
It looks good, I think that the diagram will help new developers better understand the whole concept. One thing - The current structure: |
|
Good point, feel free to add it :) |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
hey @TC-MO pls can you give it a final review and potentially merge it? |
TC-MO
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.
Just one final things otherwise LGTM
|
|
||
| Ready to start? Check out the [Actor development documentation](/platform/actors/development). | ||
|
|
||
| ## Running Actors |
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 makes sense to unify the style of heading to not use gerunds
| ## Running Actors | |
| ## Run Actors |
|
I'll go ahead and merge to not keep that PR with such a small issue, and adjust it myself in separate PR |
|
thanks! |
Improved copy about Actors - reused some copy from Actor whitepaper Small improvements. --------- Co-authored-by: Michał Olender <[email protected]> Co-authored-by: Patrik Braborec <[email protected]>
Improved copy about Actors - reused some copy from Actor whitepaper
Small improvements.